#5896: [with patch, needs trivial review] Documentation fix for lcalc interface.
---------------------------+------------------------------------------------
 Reporter:  craigcitro     |       Owner:  craigcitro
     Type:  defect         |      Status:  new       
 Priority:  minor          |   Milestone:  sage-3.4.2
Component:  number theory  |    Keywords:            
---------------------------+------------------------------------------------

Comment(by cremona):

 Minor quibble:  in the lcalc docs it says "Notice that with the --twist-
 quadratic option one is specifying the discriminant
 which can be negative, while with the --twist-primitive option one is
 specifying the conductor which should be positive. "   So one could argue
 that the character (-3/-) has discriminant -12 and conductor 12.

 Since users might expect similar, would it be better to change "conductor
 `N`" to "discriminant `D`" in both places in the patch?

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