#11920: Sympow needs specify -ffp-contract=on and should create datafiles
------------------------------+---------------------------------------------
   Reporter:  jdemeyer        |          Owner:  tbd                            
             
       Type:  defect          |         Status:  needs_work                     
             
   Priority:  major           |      Milestone:  sage-4.7.3                     
             
  Component:  packages        |       Keywords:                                 
             
Work_issues:                  |       Upstream:  None of the above - read trac 
for reasoning.
   Reviewer:  Leif Leonhardy  |         Author:  Jeroen Demeyer                 
             
     Merged:                  |   Dependencies:                                 
             
------------------------------+---------------------------------------------

Comment(by leif):

 FWIW, both with the p9 (with `-O3`, and '''without''' `-mno-fused-madd`,
 which is AFAIK the default) and with the p10 (with `-mno-fused-madd`) all
 (long) tests in

  `sage/lfunctions/sympow.py`

 pass.  So you should probably change your check to detect whether the
 doubles are 64-bit / have a 53-bit mantissa.


 Only the optional ones fail (differently); one has to compute extra data
 (with PARI/GP) for these anyway.  With the p10 I get:
 {{{
 sage -t -long -optional "devel/sage/sage/lfunctions/sympow.py"
 **********************************************************************
 File ".../sage/lfunctions/sympow.py", line 127:
     sage: a = sympow.L(EllipticCurve('11a'), 2, 16); a   # optional
 Expected:
     '1.057599244590958E+00'
 Got:
     '-4.967915042710588E+11'
 **********************************************************************
 File ".../sage/lfunctions/sympow.py", line 129:
     sage: RR(a)                    # optional -- requires precomputations
 Expected:
     1.05759924459096
 Got:
     -4.96791504271059e11
 **********************************************************************
 1 items had failures:
    2 of   5 in __main__.example_4
 ***Test Failed*** 2 failures.
 For whitespace errors, see the file /home/leif/.sage//tmp/sympow_52568.py
          [12.7 s]
 }}}

 Fairly loud noise... :-)

 (Might be that the data in the "manually" computed file is just wrong.)

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11920#comment:13>
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