Skip to content

Add run metadata and make submission metadata non-optional - #51

Merged
arav-agarwal2 merged 3 commits into
mainfrom
feat/add-run-metatadata-checks
Jun 26, 2026
Merged

Add run metadata and make submission metadata non-optional#51
arav-agarwal2 merged 3 commits into
mainfrom
feat/add-run-metatadata-checks

Conversation

@arav-agarwal2

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

Copy link
Copy Markdown

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

@anandhu-eng anandhu-eng left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

arav-agarwal2 and others added 2 commits June 26, 2026 17:06
The merge of main into this branch left the valid_standardized fixture
half-merged: model-name-validation (from main) renamed the model dir to
llama3_1-8b and required system_desc.model_name, while this branch's
run-metadata work added run_metadata.json under the old llama3-70b dir.
The result was two model dirs, a None model_name, and a stale model_id.

Reconcile into one coherent model:
- move run_metadata.json into the llama3_1-8b point dirs
- set systems model_name/model_id to "llama3.1-8b"
- drop the stale llama3-70b dir
- fix import ordering in test_builder.py from the merge

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@arav-agarwal2
arav-agarwal2 merged commit 8bb27d4 into main Jun 26, 2026
7 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants