#16836: __neg__ fails in CartesianProduct of CombinatorialFreeModule
-------------------------+-------------------------------------------------
Reporter: | Owner:
cnassau | Status: needs_review
Type: | Milestone: sage-6.4
defect | Resolution:
Priority: major | Merged in:
Component: | Reviewers:
categories | Work issues:
Keywords: | Commit:
Authors: | ae2a8c32b5071250a03c623832041292e77652d2
Christian Nassau | Stopgaps:
Report Upstream: N/A |
Branch: |
public/16836 |
Dependencies: |
-------------------------+-------------------------------------------------
Changes (by vdelecroix):
* commit: 27861148fe1ec16070603023e9a9fb643f9d6698 =>
ae2a8c32b5071250a03c623832041292e77652d2
* branch: u/cnassau/16836.2 => public/16836
Comment:
Hello Christian,
I made a small commit which forces a cast of each factor. In that case you
get a `ValueError` when you try to negate `(1, 42, 1.)` in `ZZ x NN x RR`.
Actually, wouldn't it be better that doing a negation convert it to an
element of `ZZ x ZZ x RR`. What do you think?
Best,
Vincent
PS: it would be cool if you worked on the last beta release (now
`6.7.beta1`). It helps preventing merge conflicts and it avoids switching
between Sage versions. Do not change it now! But when you restart the
implementation like you did, just start it on the last beta.
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=ae2a8c32b5071250a03c623832041292e77652d2
ae2a8c3]||{{{Trac 16836: cast each factor explicitely}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/16836#comment:9>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
