Opened 12 years ago
Closed 11 years ago
#9162 closed defect (duplicate)
cygwin/ARM:pynac.pyx use double precision special functions instead of long double (REVISITED)
Reported by: | William Stein | Owned by: | tbd |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | porting: Cygwin | Keywords: | |
Cc: | Julien Puydt | Merged in: | |
Authors: | Reviewers: | Burcin Erocal | |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
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
Change History (10)
comment:1 Changed 12 years ago by
Summary: | cygwin: → cygwin:pynac.pyx use double precision special functions instead of long double (REVISITED) |
---|
comment:2 Changed 12 years ago by
comment:3 Changed 11 years ago by
Somewhat miraculously, this passes tests on a recent build of mine on XP. Perhaps due to using mpmath now or something?
comment:4 Changed 11 years ago by
Priority: | critical → major |
---|
comment:5 Changed 11 years ago by
But this, once again, does not pass when done by hand. What is going on?
comment:6 Changed 11 years ago by
exactly the same problem also pops up on ARM running Ubuntu 11.10.
comment:7 Changed 11 years ago by
Cc: | Julien Puydt added |
---|---|
Summary: | cygwin:pynac.pyx use double precision special functions instead of long double (REVISITED) → cygwin/ARM:pynac.pyx use double precision special functions instead of long double (REVISITED) |
comment:8 Changed 11 years ago by
Milestone: | sage-5.0 → sage-duplicate/invalid/wontfix |
---|---|
Status: | new → needs_review |
#12449 contains a patch to fix. We should close this as a duplicate.
comment:9 Changed 11 years ago by
Status: | needs_review → positive_review |
---|
comment:10 Changed 11 years ago by
Description: | modified (diff) |
---|---|
Resolution: | → duplicate |
Reviewers: | → Burcin Erocal |
Status: | positive_review → closed |
Note: See
TracTickets for help on using
tickets.
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...