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.

Sign up for GitHub

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

Jump to bottom

Remove AutoShardingResult in favor of a boolean now that the value kModuleUnchangedNoShardingPerformed of the enum is unused, effectively making it a boolean. Also simplified away some dead code. #17874

Merged
merged 1 commit into from
Oct 2, 2024
+17 −35

Remove AutoShardingResult in favor of a boolean now that the value kM…

13e486d
Select commit
Loading
Failed to load commit list.
Merged

Remove AutoShardingResult in favor of a boolean now that the value kModuleUnchangedNoShardingPerformed of the enum is unused, effectively making it a boolean. Also simplified away some dead code. #17874

Remove AutoShardingResult in favor of a boolean now that the value kM…
13e486d
Select commit
Loading
Failed to load commit list.
Google CLA / cla/google succeeded Oct 2, 2024 in 2s

✅ All contributors are covered under a CLA with Google

See https://cla.developers.google.com/ for more info about Google's Contributor License Agreement (CLA).

ℹ️ Googlers: Go here to view more details and manage scans for this pull request.

Details

The following contributors were found for this pull request:

✅ 13e486d PR Opener: @copybara-service[bot]
✅ 13e486d Author: @Google-ML-Automation <googl**********ation​@google.com>

(Only the first commit for a unique contributor is listed.)

View more details on Google CLA