Skip to content
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

[shard_map] fix jax.debug.print/callback under partial auto #25887

Merged
merged 1 commit into from
Feb 15, 2025

Conversation

mattjj
Copy link
Collaborator

@mattjj mattjj commented Jan 14, 2025

This change makes jax.debug.print work under a partial-auto shard_map. I think it's an alternative to #25705 which doesn't need openxla/xla#20944.

While this makes jax.debug.print work today, corresponding with the semantics of today, #25842 points to deeper improvements we still want to make.

@mattjj mattjj requested a review from pschuh January 14, 2025 23:41
@mattjj mattjj force-pushed the partial-auto-something branch 2 times, most recently from 6147eab to 64b0bb6 Compare January 14, 2025 23:42
@mattjj mattjj changed the title Partial auto something [shard_map] fix jax.debug.print/callback under partial auto Jan 15, 2025
@mattjj mattjj force-pushed the partial-auto-something branch 2 times, most recently from 6bf215b to 1039edd Compare January 15, 2025 01:11
@mattjj mattjj self-assigned this Jan 15, 2025
@mattjj mattjj added the pull ready Ready for copybara import and testing label Jan 15, 2025
@mattjj mattjj added pull ready Ready for copybara import and testing and removed pull ready Ready for copybara import and testing labels Jan 22, 2025
@mattjj mattjj added pull ready Ready for copybara import and testing and removed pull ready Ready for copybara import and testing labels Jan 30, 2025
@mattjj mattjj added pull ready Ready for copybara import and testing and removed pull ready Ready for copybara import and testing labels Feb 10, 2025
@mattjj mattjj force-pushed the partial-auto-something branch from 1039edd to 3681960 Compare February 15, 2025 00:24
@copybara-service copybara-service bot merged commit df135d2 into jax-ml:main Feb 15, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pull ready Ready for copybara import and testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants