-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TestParseDecimal/5.1 no longer passes #115
Comments
Thanks for the report. Can't figure out what's wrong here, with Absolute: Relative: For both cases the test should pass (as it does on my computer), not sure what is wrong... |
Pushed out a change in the test package, maybe that fixes it? |
Thanks, it looks like 2.7.1 works, and I didn't need to use the intermediate commit of the Edit: And also 2.7.4 works, without the change to |
This is probably dependent on architecture in some way, or floating-point optimizations. I am running on a now somewhat older x86_64 though, so it's not too esoteric.
This occurs with the older
github.com/tdewolff/test
1.0.9 that had absolute comparisons, or with the newer 1.0.10 that has relative comparisons.The text was updated successfully, but these errors were encountered: