-
Notifications
You must be signed in to change notification settings - Fork 40
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
[BUG] Unit tests failures #169
Comments
forgot to add: this is for the |
Attaching the output of the failures: This seems to be a regression which wasn't covered in our CI because we only test the intersection if |
I've diagnosed the cause of the issue, will have a patch incoming shortly and an immediately release once patched. This never should have made it to production, not having this specific case covered in CI is a major oversight on my part and I'll make sure this case is always covered in CI going forward. |
Thanks for being so responsive :) |
Sorry I haven't forgotten about this and am working on it. Other open source work and my day job have gotten in the way. This is still my highest priority for this crate at the moment. |
Sorry for the massive delay on this, I've implemented a large number of patches and optimizations and completed this. I will be drafting up a release shortly. |
Published as of version 1.0.3. |
thanks a lot :) |
Description
cargo test --no-default-features --features radix
fails two unit tests:i128_proptest_radix and
u128_proptest_radix` on my amd64 machine.Prerequisites
Here are a few things you should provide to help me understand the issue:
rustc 1.81.0 (eeb90cda1 2024-09-04) (Arch Linux rust 1:1.81.0-1)
--no-default-features --features radix
Test case
See above, it's the existing unit tests.
Additional Context
cargo test --no-default-features --features power-of-two
also have some failing tests:binary_test
andu128toa_proptest
The text was updated successfully, but these errors were encountered: