#3999: [with patch, needs doctests] Wrapper class to treat additive groups as
multiplicative goups
------------------------------+---------------------------------------------
Reporter: robertwb | Owner: somebody
Type: defect | Status: new
Priority: major | Milestone: sage-3.2.2
Component: basic arithmetic | Resolution:
Keywords: |
------------------------------+---------------------------------------------
Comment (by cremona):
Replying to [comment:6 robertwb]:
> The bitrot was due to #4310 and #4175, the patch has been updated.
>
> Thre needs to be more doctests.
>
> {{{
> sage/groups/multiplicative_wrapper.pyx
> ERROR: Please define a s == loads(dumps(s)) doctest.
> SCORE sage/groups/multiplicative_wrapper.pyx: 20% (5 of 24)
> ...
> }}}
>
> I'll hold off until I see John Cremona's patch. Could you post it, even
if it's not done yet? I'm not sure I like the idea of {{{a+b}}} just
working for any multiplicative group though.
OK, will do. a+b will not work for multiplicative groups: it will give a
NotImplementedError, while a*b will work. And vive versa if the group was
created as multiplicative (which will be the old, default behaviour).
>
> Most of my patch is not about coercion, but anyone doesn't understand
how coercion is working I'm the one to blame until I at the very least put
out some good documentation.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/3999#comment:7>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
