#8357: Doctesting ell_rational_field.py leaves a file PRIMES in the current
directory
------------------------------------------+---------------------------------
   Reporter:  mpatel                      |       Owner:  cremona     
       Type:  defect                      |      Status:  needs_review
   Priority:  major                       |   Milestone:  sage-4.3.4  
  Component:  elliptic curves             |    Keywords:              
     Author:  John Cremona, Mitesh Patel  |    Upstream:  N/A         
   Reviewer:                              |      Merged:              
Work_issues:                              |  
------------------------------------------+---------------------------------
Changes (by newvalueoldvalue):

  * status:  new => needs_review
  * milestone:  => sage-4.3.4
  * author:  => John Cremona, Mitesh Patel


Old description:

> From [http://groups.google.com/group/sage-
> devel/browse_thread/thread/976cf8ecb4896e7c John Palmieri]:
> {{{
> When I run doctests on the file ell_rational_field.py, I end up with a
> small file called PRIMES in the current directory. This shouldn't
> happen: running doctests shouldn't produce files in a non-temporary
> directory.  However, I can't figure out how this file gets there.  In
> particular, if I delete the following doctests from the rank method,
> then the file is not produced:
>
> {{{
> diff -r 23241bd151e3 sage/schemes/elliptic_curves/
> ell_rational_field.py
> --- a/sage/schemes/elliptic_curves/ell_rational_field.py        Thu
> Feb 18 14:25:25 2010 -0800
> +++ b/sage/schemes/elliptic_curves/ell_rational_field.py        Wed
> Feb 24 20:37:09 2010 -0800
> @@ -1679,14 +1679,6 @@
>              sage: E.minimal_model().rank()
>              1
>
> -        A large example where mwrank doesn't determine the result
> with certainty::
> -
> -            sage: EllipticCurve([1,0,0,0,37455]).rank(proof=False)
> -            0
> -            sage: EllipticCurve([1,0,0,0,37455]).rank(proof=True)
> -            Traceback (most recent call last):
> -            ...
> -            RuntimeError: Rank not provably correct.
>          """
>          if proof is None:
>              from sage.structure.proof.proof import get_flag
>
> }}}
>
> If I run these by hand from the command line, though, I don't see the
> file PRIMES anywhere.  Can someone familiar with the elliptic curves
> code open a ticket, chase this down, and fix it?
> }}}
>
> This is a follow-up to #7575.  Please see [comment:ticket:7575:24 comment
> 24+] for some progress.

New description:

 From [http://groups.google.com/group/sage-
 devel/browse_thread/thread/976cf8ecb4896e7c John Palmieri]:
 {{{
 When I run doctests on the file ell_rational_field.py, I end up with a
 small file called PRIMES in the current directory. This shouldn't
 happen: running doctests shouldn't produce files in a non-temporary
 directory.
 [...]
 }}}

 This is a follow-up to #7575.  Please see [comment:ticket:7575:24 comment
 24+] for some progress.

 A new spkg is available at

  *
 http://sage.math.washington.edu/home/mpatel/trac/8357/eclib-20080310.p10.spkg

 It includes

  * The quick fix mentioned below
 [attachment:trac_8357-suppress_PRIMES.patch]
  * Parallel build fixes from #8306:
 [attachment:trac_8357-eclib_makefiles.patch],
 [attachment:trac_8357-spkg.patch]

 '''Note to release manager:''' Merge only the spkg.

--

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