Ticket #4707 (closed defect: fixed)

Opened 1 month ago

Last modified 1 month ago

[with patch; positive review] magma/sage interface -- another trivial easy-to-fix failure hopefully

Reported by: was Assigned to: was
Priority: major Milestone: sage-3.2.2
Component: interfaces Keywords:
Cc:

Description 

sage: E = EllipticCurve(GF(25,'a'), [0,0,1,4,0])
sage: magma(E)
boom

Attachments

trac_4707.patch (2.8 kB) - added by was on 12/04/2008 06:54:35 PM.
trac_4707_tiny_followup.patch (1.0 kB) - added by was on 12/06/2008 03:49:26 PM.

Change History

12/04/2008 06:54:35 PM changed by was

  • attachment trac_4707.patch added.

12/04/2008 06:55:03 PM changed by was

  • summary changed from magma/sage interface -- another trivial easy-to-fix failure hopefully to [with patch; needs review (EASY!)] magma/sage interface -- another trivial easy-to-fix failure hopefully.

12/06/2008 03:49:26 PM changed by was

  • attachment trac_4707_tiny_followup.patch added.

12/06/2008 03:49:58 PM changed by ncalexan

  • summary changed from [with patch; needs review (EASY!)] magma/sage interface -- another trivial easy-to-fix failure hopefully to [with patch, with postitive review] magma/sage interface -- another trivial easy-to-fix failure hopefully.

Works for me. Apply both patches. We will open a separate ticket for the corresponding magma -> sage conversion.

12/10/2008 08:58:07 PM changed by was

  • summary changed from [with patch, with postitive review] magma/sage interface -- another trivial easy-to-fix failure hopefully to [with patch, with positive review] magma/sage interface -- another trivial easy-to-fix failure hopefully.

fix typo in title

12/10/2008 08:58:22 PM changed by was

  • summary changed from [with patch, with positive review] magma/sage interface -- another trivial easy-to-fix failure hopefully to [with patch; positive review] magma/sage interface -- another trivial easy-to-fix failure hopefully.

12/11/2008 03:09:55 AM changed by mabshoff

  • status changed from new to closed.
  • resolution set to fixed.

Merged both patches in Sage 3.2.2.alpha2