-
-
Notifications
You must be signed in to change notification settings - Fork 31.5k
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
bpo-37596: Update test_deterministic_sets
to correctly handle different string hash algorithms
#28147
Conversation
🤖 New build scheduled with the buildbot fleet by @brandtbucher for commit e72a25b 🤖 If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again. |
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.
So did you just try different strings until you found one that hashes the same under fnv and siphash24?? That's amazing. (I didn't even know we had all this machinery -- all I know is that we have "hash randomization" and that you can force the seed by setting an env var. :-)
No, I just needed to find one that hashes in such a way that sets containing them have different orders under
Same, I learned a lot about this stuff today! |
That buildbot's been failing for a while with that same error. I'll open an issue for it. |
@brandtbucher: Please replace |
|
https://bugs.python.org/issue37596