Skip to content

fix: save datasets selected by run mode - #548

Draft
TeoSlayer wants to merge 1 commit into
mlcommons:mainfrom
TeoSlayer:fix/run-mode-config
Draft

TeoSlayer wants to merge 1 commit into
mlcommons:mainfrom
TeoSlayer:fix/run-mode-config

Conversation

@TeoSlayer

@TeoSlayer TeoSlayer commented Oct 28, 2026 •

Copy link
Copy Markdown

Change

Save only datasets selected by --mode. Keep runtime config and secret redaction intact.

Closes #420.

Checks

  • All three modes tested; selected modes fail before the fix.
  • 2,234 unit tests pass; 27 skip.
  • One macOS affinity test fails on both branches.
  • Linux mypy passes. Other hooks pass; macOS mypy flags three existing Linux-only calls.

CI has a base timeout failure. Audit fix: #551. Review requests lack fork permission.

@github-actions

Copy link
Copy Markdown

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/normal PR Review Policy: <=500 non-test lines & <=20 files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: output config.yaml should respect --mode flag

1 participant