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: |
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 11 years ago by
- 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
comment:3 Changed 10 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 10 years ago by
- Priority changed from critical to major
comment:5 Changed 10 years ago by
But this, once again, does not pass when done by hand. What is going on?
comment:6 Changed 9 years ago by
exactly the same problem also pops up on ARM running Ubuntu 11.10.
comment:7 Changed 9 years ago by
- 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
- 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
- Status changed from needs_review to positive_review
comment:10 Changed 9 years ago by
- 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.
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...