#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:                    |  
------------------------------+---------------------------------------------
Changes (by robertwb):

  * summary:  [with patch, needs work] Wrapper class to treat additive
              groups as multiplicative goups => [with patch,
              needs doctests] Wrapper class to treat additive
              groups as multiplicative goups

Comment:

 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.

 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:6>
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to