Skip to content

Conversation

juanitorduz
Copy link
Collaborator

Ensures compatibility with https://github.com/google/flax/releases/tag/v0.11.2

The tests were failing in https://github.com/pyro-ppl/numpyro/actions/runs/17463007632/job/49592472608

FAILED test/contrib/test_module.py::test_nnx_state_dropout_smoke[batchnorm-dropout] - flax.errors.TraceContextError: Cannot mutate BatchStat from a different trace level (https://flax.readthedocs.io/en/latest/api_reference/flax.errors.html#flax.errors.TraceContextError)
--------------------
For simplicity, JAX has removed its internal frames from the traceback of the following exception. Set JAX_TRACEBACK_FILTERING=off to include these.
FAILED test/contrib/test_module.py::test_nnx_state_dropout_smoke[batchnorm-no_dropout] - flax.errors.TraceContextError: Cannot mutate BatchStat from a different trace level (https://flax.readthedocs.io/en/latest/api_reference/flax.errors.html#flax.errors.TraceContextError)
--------------------
For simplicity, JAX has removed its internal frames from the traceback of the following exception. Set JAX_TRACEBACK_FILTERING=off to include these.
= 2 failed, 506 passed, 66 skipped, 4 xfailed, 36 warnings in 654.96s (0:10:54)

Copy link
Member

@fehiepsi fehiepsi left a comment

Choose a reason for hiding this comment

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

Thanks for the fix @juanitorduz!

@juanitorduz
Copy link
Collaborator Author

I will rebase and remove the xfail once #2069 is merged

@fehiepsi
Copy link
Member

fehiepsi commented Sep 5, 2025

Hi @juanitorduz, I just merged the other PR.

@juanitorduz
Copy link
Collaborator Author

Hi @juanitorduz, I just merged the other PR.

I just rebased and removed xfail

@fehiepsi fehiepsi merged commit 68d86dc into pyro-ppl:master Sep 6, 2025
9 checks passed
@juanitorduz juanitorduz deleted the nnx_merge_11_2 branch September 29, 2025 12:02
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.

2 participants