Opened 6 years ago

Closed 5 years ago

#16713 closed defect (fixed)

Random failure in lseries_ell.py

Reported by: vbraun Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: elliptic curves Keywords: random_fail pari lcalc
Cc: jdemeyer, leif, was Merged in:
Authors: Reviewers: Jeroen Demeyer
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

This is a somewhat regular occurrence on mod: 

sage -t --long src/sage/schemes/elliptic_curves/lseries_ell.py
**********************************************************************
File "src/sage/schemes/elliptic_curves/lseries_ell.py", line 243, in sage.schemes.elliptic_curves.lseries_ell.Lseries_ell.zeros
Failed example:
    E.lseries().zeros(2)
Expected:
    [0.000000000, 5.00317001]
Got:
      ***   Warning: not enough memory, new stack 1000000000.
    [0.000000000, 5.00317001]

There seems to be plenty of free space. Obviously there is enough memory to finish the computation and arrive at the correct result, so there shouldn't be any warning printed. Related tickets that patched up similar issues:

  • Trac #11985: lcalc is too verbose (mysterious "Warning: new stack size")
  • Trac #14470: Relax PARI warnings

Change History (9)

comment:1 Changed 6 years ago by vbraun

  • Cc jdemeyer leif was added
  • Keywords pari lcalc added

comment:2 Changed 6 years ago by leif

Well, 8.04 is out of LTS... ;-)

Did anybody else face this on another system / platform?

comment:3 Changed 6 years ago by vbraun

I'll keep an eye out for it... Still, it doesn't look like a compiler bug.

comment:4 Changed 6 years ago by jdemeyer

Let's hope that this problem disappears with #15767.

comment:5 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:6 Changed 5 years ago by jdemeyer

Has this happened recently? We upgraded PARI several times in the mean time...

comment:7 Changed 5 years ago by jdemeyer

  • Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
  • Reviewers set to Jeroen Demeyer
  • Status changed from new to needs_review

comment:8 Changed 5 years ago by jdemeyer

  • Status changed from needs_review to positive_review

comment:9 Changed 5 years ago by vbraun

  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.