#4822: [with patch, under review] Tweak to the error message for EllipticCurve
-----------------------------+----------------------------------------------
 Reporter:  ljpk             |       Owner:  davidloeffler
     Type:  enhancement      |      Status:  new          
 Priority:  minor            |   Milestone:  sage-4.1.2   
Component:  elliptic curves  |    Keywords:               
 Reviewer:                   |      Author:               
   Merged:                   |  
-----------------------------+----------------------------------------------

Comment(by wuthrich):

 The patch is ok (so far only tested without the optional package). I will
 test it with it later today.

 Maybe while we are at it I could suggest that the same change is done to
 {{{cremona_label()}}}.
 This is without the optional package:

 {{{
 sage: E= EllipticCurve([3,10])
 sage: E.cremona_label()
 ---------------------------------------------------------------------------
 RuntimeError                              Traceback (most recent call
 last)

 /home/chrigu/.sage/temp/linux_ljo8/12909/_home_chrigu__sage_init_sage_0.py
 in <module>()

 /usr/local/sage/local/lib/python2.6/site-
 packages/sage/schemes/elliptic_curves/ell_rational_field.pyc in
 cremona_label(self, space)
    3305                 X = self.database_curve()
    3306             except RuntimeError:
 -> 3307                 raise RuntimeError, "Cremona label not known for
 %s."%self
    3308             self.__cremona_label = X.__cremona_label
    3309             return self.cremona_label(space)

 RuntimeError: Cremona label not known for Elliptic Curve defined by y^2 =
 x^3 + 3*x + 10 over Rational Field.
 sage: E.conductor()
 44928
 }}}

 Chris.

 ps : 'I don't know if there is an official 'under review'.

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