-
Notifications
You must be signed in to change notification settings - Fork 193
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
Relax absolute tolerance for failing tests involving chex.assert_trees_all_close. #1069
Relax absolute tolerance for failing tests involving chex.assert_trees_all_close. #1069
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
Thanks @carlosgmartin ! I don't see when or where these tests broke. Could you point out the reason for this change? |
@vroulet Here's the output I get from running tests on my machine (after applying #1068 and #1071):
Do you get the same output from running tests on your machine? |
Indeed, I get the error too. I don't understand why it's not been caught on the tests in github.
and for the linesearch test:
I checked, the tests pass with these changes. |
The doctest of _dog also fails on my end.
(This algorithm needs to be completed with a proper evaluation function and fuse with a similar one in the alias/transform files.) |
1017127
to
ddbd589
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small linting comment
ddbd589
to
ff12a82
Compare
No description provided.