Skip to content

fix: do not use atol=np.inf in a jax test and xfail sorting test which fails#3542

Merged
ikrommyd merged 3 commits intomainfrom
ikrommyd/do-not-use-infinity-atol-and-comment-out-bad-test
Jun 13, 2025
Merged

fix: do not use atol=np.inf in a jax test and xfail sorting test which fails#3542
ikrommyd merged 3 commits intomainfrom
ikrommyd/do-not-use-infinity-atol-and-comment-out-bad-test

Conversation

@ikrommyd
Copy link
Copy Markdown
Collaborator

This has to do with numpy 2.3 which raises a warning about invalid atol if it's set to infinity and also the fact that this test is not right and jax and awkward give different values. See #3541

@ikrommyd ikrommyd changed the title fix: do not use infinity for atol in a jax test and comment out sorting test which fails fix: do not use atol=np.inf in a jax test and comment out sorting test which fails Jun 13, 2025
@ikrommyd ikrommyd requested a review from ianna June 13, 2025 11:39
Copy link
Copy Markdown
Member

@ianna ianna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ikrommyd - Thanks! Please uncomment the sort tests. It is better to have them on record to remind us. Thanks!

@ikrommyd ikrommyd changed the title fix: do not use atol=np.inf in a jax test and comment out sorting test which fails fix: do not use atol=np.inf in a jax test and xfail out sorting test which fails Jun 13, 2025
@ikrommyd ikrommyd changed the title fix: do not use atol=np.inf in a jax test and xfail out sorting test which fails fix: do not use atol=np.inf in a jax test and xfail sorting test which fails Jun 13, 2025
Copy link
Copy Markdown
Member

@ianna ianna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ikrommyd - Great! Thanks for fixing it. Please go ahead and merge it if you finished with it. Thanks!

@ikrommyd ikrommyd merged commit 939b46c into main Jun 13, 2025
43 of 44 checks passed
@ikrommyd ikrommyd deleted the ikrommyd/do-not-use-infinity-atol-and-comment-out-bad-test branch June 13, 2025 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants