Opened 8 years ago

Closed 8 years ago

#14470 closed defect (fixed)

Relax PARI warnings

Reported by: jdemeyer Owned by: mvngu
Priority: blocker Milestone: sage-5.9
Component: doctest coverage Keywords: new stack size not enough memory
Cc: Merged in: sage-5.9.rc0
Authors: Jeroen Demeyer Reviewers: Leif Leonhardy
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

Sometimes:

sage -t --long devel/sage/sage/schemes/elliptic_curves/lseries_ell.py
**********************************************************************
File "devel/sage/sage/schemes/elliptic_curves/lseries_ell.py", line 226, in sage.schemes.elliptic_curves.lseries_ell.Lseries_ell.zeros
Failed example:
    E.lseries().zeros(2)
Expected:
      ***   Warning: new stack size = ...
    [0.000000000, 5.00317001]
Got:
      ***   Warning: not enough memory, new stack 1000000000.
      ***   Warning: new stack size = 1028720 (0.981 Mbytes).
    [0.000000000, 5.00317001]
**********************************************************************
File "devel/sage/sage/schemes/elliptic_curves/lseries_ell.py", line 230, in sage.schemes.elliptic_curves.lseries_ell.Lseries_ell.zeros
Failed example:
    a = E.lseries().zeros(20)             # long time
Expected:
      ***   Warning: new stack size = ...
Got:
      ***   Warning: not enough memory, new stack 1000000000.
      ***   Warning: new stack size = 1033680 (0.986 Mbytes).
**********************************************************************

Attachments (1)

14470_pari_warn.patch (4.3 KB) - added by jdemeyer 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by jdemeyer

comment:1 Changed 8 years ago by leif

  • Keywords new stack size not enough memory added
  • Reviewers set to Leif Leonhardy
  • Status changed from new to needs_review

Looks like you catched all instances, and doctests still pass. (I didn't get such failures previously though.)

comment:2 Changed 8 years ago by leif

  • Status changed from needs_review to positive_review

comment:3 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.9.rc0
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.