-
Notifications
You must be signed in to change notification settings - Fork 37
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
Move FLOAT_CMP upstream and/or combine with numpy/scipy? #55
Comments
I'd be in favor, especially given the fact that we recently discovered some issues with the way that |
This is something that would be very useful. I can look into it, definitely. 😄 |
Pytest has added |
I think we decided that Superseded by #199 |
Let's keep this open for the moment, since perhaps |
As noted originally in astropy/astropy#6554, numpy was thinking about their own version of something like
FLOAT_CMP
and ended up adapting scipy'srefguide
[1] for its docstring testing. It includes float comparisons and "numbers may vary". We should check whether we can also use this, to make the ecosystem a bit more uniform. In this respect, probably good to remember that this was put also to python-ideas by @kdavies4 with Coghlan concluding it was a good idea [2]. (See also numpy/numpy#9415 (comment))[1] https://github.com/scipy/scipy/blob/master/tools/refguide_check.py
[2] https://mail.python.org/pipermail/python-ideas/2014-August/028559.html
The text was updated successfully, but these errors were encountered: