Skip to content

Add the optional workload_id flag#3299

Merged
copybara-service[bot] merged 2 commits intoAI-Hypercomputer:mainfrom
CIeNET-International:maxtxt/user/dora/benchmarks_workload_id
Mar 12, 2026
Merged

Add the optional workload_id flag#3299
copybara-service[bot] merged 2 commits intoAI-Hypercomputer:mainfrom
CIeNET-International:maxtxt/user/dora/benchmarks_workload_id

Conversation

@RUEI4341
Copy link
Contributor

@RUEI4341 RUEI4341 commented Mar 3, 2026

Description

Currently, the pw_mcjax_benchmark_recipe ignores user-specified workload names, instead defaulting to an auto-generated string composed of the username, model name, and a random 3-character suffix (e.g., roo-pw-default-1-ogo).

This PR introduces an optional --workload_id flag. When provided, the system bypasses the random name generation logic and uses the user's specific string for the XPK workload name and the MaxText run_name.,
This allows users to align XPK workloads with specific CI/CD jobs, internal experiment tracking IDs, and ensures predictable GCS output paths.

Tests

https://paste.googleplex.com/5792509692346368

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • I have performed a self-review of my code. For an optional AI review, add the gemini-review label.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run end-to-end tests tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in our documentation.

@RUEI4341 RUEI4341 force-pushed the maxtxt/user/dora/benchmarks_workload_id branch from 13bf82e to f885131 Compare March 12, 2026 02:30
@copybara-service copybara-service bot merged commit c0b15ef into AI-Hypercomputer:main Mar 12, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants