Skip to content

Make Maxdiffusion compatile w/ JAX 0.7.0 and latest orbax version#211

Merged
coolkp merged 3 commits intomainfrom
rbierneni-orbax-abstractlogger-fix
Jul 28, 2025
Merged

Make Maxdiffusion compatile w/ JAX 0.7.0 and latest orbax version#211
coolkp merged 3 commits intomainfrom
rbierneni-orbax-abstractlogger-fix

Conversation

@Rohan-Bierneni
Copy link
Copy Markdown
Collaborator

@Rohan-Bierneni Rohan-Bierneni commented Jul 26, 2025

This fix allows us to use the latest version of orbax. Use these changes to test maxdiffusion + jax 0.7.0 + python 3.12. Only merge these changes once unit tests and XLML tests pass.

A maxdiffusion docker image with these changes already made is here and a maxdiffusion without the orbax changes is here.

Logs from failing XLML DAG with updated orbax API: https://paste.googleplex.com/5559411468402688#l=348

Update:
Tested that a sdxl workload now passes using the image w/ JAX 0.7.0 and latest orbax.

https://screenshot.googleplex.com/8ga2wtDFyeWkiZJ
https://screenshot.googleplex.com/6BfijSpUf6Me4rS

@github-actions
Copy link
Copy Markdown

@entrpn entrpn self-requested a review July 28, 2025 18:30
entrpn
entrpn previously approved these changes Jul 28, 2025
@github-actions
Copy link
Copy Markdown

@Rohan-Bierneni Rohan-Bierneni changed the title [Do not Merge] Unpinned orbax and updated orbax logger API Make Maxdiffusion compatile w/ JAX 0.7.0 and latest orbax version Jul 28, 2025
entrpn
entrpn previously approved these changes Jul 28, 2025
@Rohan-Bierneni Rohan-Bierneni marked this pull request as ready for review July 28, 2025 19:45
@github-actions
Copy link
Copy Markdown

@coolkp coolkp merged commit e556ca1 into main Jul 28, 2025
3 of 4 checks passed
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