Ticket #4391 (closed defect: fixed)

Opened 17 months ago

Last modified 17 months ago

[with patch, with positive review] Sage 3.1.4: optional doctest failure in sage/schemes/elliptic_curves/ell_finite_field.py

Reported by: mabshoff Owned by: mabshoff
Priority: major Milestone: sage-3.2
Component: doctest Keywords:
Cc: Author(s):
Report Upstream: Reviewer(s):
Merged in: Work issues:

Description

sage -t -long -optional devel/sage/sage/schemes/elliptic_curves/ell_finite_field.py
**********************************************************************
File "/scratch/mabshoff/release-cycle/sage-3.1.3.final/tmp/ell_finite_field.py", line 102:
    sage: magma(E) # optional -- requires Magma
Exception raised:
    Traceback (most recent call last):
      File "/scratch/mabshoff/release-cycle/sage-3.1.3.final/local/lib/python2.5/doctest.py", line 1228, in __run
        compileflags, 1) in test.globs
      File "<doctest __main__.example_4[3]>", line 1, in <module>
        magma(E) # optional -- requires Magma###line 102:
    sage: magma(E) # optional -- requires Magma
    NameError: name 'E' is not defined
**********************************************************************

Attachments

trac_4391.patch Download (1.1 KB) - added by mabshoff 17 months ago.

Change History

Changed 17 months ago by mabshoff

  • status changed from new to assigned

Changed 17 months ago by craigcitro

  • summary changed from [with patch, needs review] Sage 3.1.4: optional doctest failure in sage/schemes/elliptic_curves/ell_finite_field.py to [with patch, with positive review] Sage 3.1.4: optional doctest failure in sage/schemes/elliptic_curves/ell_finite_field.py

Looks good.

Changed 17 months ago by mabshoff

I had to rebase t my own patch for Saeg 3.2.alpha1, oh well :)

Cheers,

Michael

Changed 17 months ago by mabshoff

Changed 17 months ago by mabshoff

  • status changed from assigned to closed
  • resolution set to fixed

Merged in Sage 3.2.alpha2

Note: See TracTickets for help on using tickets.