#5098: [with patch, with positive review, needs doctest fix] Pollard rho 
algorithm
for generic discrete logarithm
-------------------------+--------------------------------------------------
 Reporter:  ylchapuy     |        Owner:  tbd     
     Type:  enhancement  |       Status:  new     
 Priority:  minor        |    Milestone:  sage-3.3
Component:  algebra      |   Resolution:          
 Keywords:               |  
-------------------------+--------------------------------------------------
Changes (by mabshoff):

  * summary:  [with patch, with positive review] Pollard rho algorithm for
              generic discrete logarithm => [with patch, with
              positive review, needs doctest fix] Pollard rho
              algorithm for generic discrete logarithm

Comment:

 The latest patch causes the following doctest failure: From
 {{{
 File
 "/scratch/mabshoff/sage-3.3.alpha4/devel/sage/sage/rings/integer_mod.pyx",
 line 444:
    sage: sage.groups.generic.discrete_log(a,b)
 Expected:
    Traceback (most recent call last):
    ...
    ZeroDivisionError: Inverse does not exist.
 }}}
 we get to
 {{{
 ArithmeticError: multiplicative order of 4 not defined since it is not a
 unit modulo 100
 }}}

 William doesn't like either one, but he is about to comment himself.

 Cheers,

 Michael

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5098#comment:12>
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