Ticket #9162 (closed defect: duplicate)

Opened 3 years ago

Last modified 15 months ago

cygwin/ARM:pynac.pyx use double precision special functions instead of long double (REVISITED)

Reported by: was Owned by: tbd
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: porting: Cygwin Keywords:
Cc: Snark Work issues:
Report Upstream: N/A Reviewers: Burcin Erocal
Authors: Merged in:
Dependencies: Stopgaps:

Description (last modified by jdemeyer) (diff)

This is a followup to #8847, which was supposed to fix this, but simply didn't. 

sage -t  "devel/sage/sage/functions/other.py"               
**********************************************************************
File "/home/wstein/sage-4.4.3/devel/sage/sage/functions/other.py", line 475:
    sage: gamma1(float(6))
Expected:
    120.0
Got:
    119.99999999999997
**********************************************************************
1 items had failures:
   1 of  29 in __main__.example_12

See #8847 for more details. Fixed by #12449.

Change History

comment:1 Changed 3 years ago by was

  • Summary changed from cygwin: to cygwin:pynac.pyx use double precision special functions instead of long double (REVISITED)

comment:2 Changed 3 years ago by mhansen

I don't think that #8847 was supposed to fix this. #8847 was to make it so that we get 120.0 instead of 119.99999999999997 on Linux. Cygwin has always given 119.9999...

comment:3 Changed 22 months ago by kcrisman

Somewhat miraculously, this passes tests on a recent build of mine on XP. Perhaps due to using mpmath now or something?

comment:4 Changed 21 months ago by kcrisman

  • Priority changed from critical to major

comment:5 Changed 21 months ago by kcrisman

But this, once again, does not pass when done by hand. What is going on?

comment:6 Changed 16 months ago by dimpase

exactly the same problem also pops up on ARM running Ubuntu 11.10.

comment:7 Changed 16 months ago by dimpase

  • Cc Snark added
  • Summary changed from cygwin:pynac.pyx use double precision special functions instead of long double (REVISITED) to cygwin/ARM:pynac.pyx use double precision special functions instead of long double (REVISITED)

comment:8 Changed 15 months ago by burcin

  • Status changed from new to needs_review
  • Milestone changed from sage-5.0 to sage-duplicate/invalid/wontfix

#12449 contains a patch to fix. We should close this as a duplicate.

comment:9 Changed 15 months ago by dimpase

  • Status changed from needs_review to positive_review

comment:10 Changed 15 months ago by jdemeyer

  • Status changed from positive_review to closed
  • Reviewers set to Burcin Erocal
  • Resolution set to duplicate
  • Description modified (diff)
Note: See TracTickets for help on using tickets.