Opened 11 years ago

Closed 9 years ago

#9162 closed defect (duplicate)

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 Merged in:
Authors: Reviewers: Burcin Erocal
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

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 (10)

comment:1 Changed 11 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 11 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 10 years 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 10 years ago by kcrisman

  • Priority changed from critical to major

comment:5 Changed 10 years ago by kcrisman

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

comment:6 Changed 9 years ago by dimpase

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

comment:7 Changed 9 years 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 9 years ago by burcin

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

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

comment:9 Changed 9 years ago by dimpase

  • Status changed from needs_review to positive_review

comment:10 Changed 9 years ago by jdemeyer

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