Skip to content

Bring in Rules.md and YAML schema#222

Closed
FileSystemGuy wants to merge 23 commits intoTF_Modernizationfrom
FileSystemGuy-newRules
Closed

Bring in Rules.md and YAML schema#222
FileSystemGuy wants to merge 23 commits intoTF_Modernizationfrom
FileSystemGuy-newRules

Conversation

@FileSystemGuy
Copy link
Copy Markdown
Contributor

This brings in the latest version of the Rules.md file, and an early snapshot of the ongoing development of the YAML schema checker. Future Rules.md development should be done in the Modernization branch (ie: I'll stop working on it in my branch), but since @dslik is working on the Schema (based in my branch), he will need to decide how to get his work into the Modernization branch.

Expanded the directory structure requirements for CLOSED and OPEN submissions, detailing the necessary files and their organization within the submission structure.
Updated submission guidelines and directory structure requirements for OPEN and CLOSED benchmarks, including detailed validation checks and file requirements.
Remove all the text that carried over from the Submission_guidelines.md file, and start filling out the main structure of the sections of the new document.
Updated sections for validating training options and checkpointing options in the rules document.
Removed section on validating phases and its related content.
Reorganize the sections under 'Validating the Checkpointing Options' for clarity.
First drraft of a schema for the SystemDescription.yaml files created by submitters.  This is in the format required by Yamale.
Add README for submission validation checker directory
@FileSystemGuy FileSystemGuy requested a review from a team December 23, 2025 19:56
@FileSystemGuy FileSystemGuy requested a review from a team as a code owner December 23, 2025 19:56
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 23, 2025

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

@FileSystemGuy
Copy link
Copy Markdown
Contributor Author

I plan to reformat the text to include a short title when I get a chance. If this PR has not yet been merged into Modernization when I'm ready to do that work, I'll do the work in my branch and do another PR to bring it into Modernization.

Clarify the meaning of two rules.
Updated rules to clarify requirements and add description tokens for each section.
@dslik
Copy link
Copy Markdown
Contributor

dslik commented Jan 14, 2026

FYI: PR mlcommons/inference_policies#324 for proposed YAML for PSU data. We'll want to start working on integrating this into the YAML.

@FileSystemGuy
Copy link
Copy Markdown
Contributor Author

This PR adds a file to the submission checker, which is Pablo's area, and should not be in this PR. It also defines the SystemDescription_Schema.yaml, which David Slik has been working on, so we should wait for his version to come in rather than making him merge his changes into this file. The Rules.md file will be merged via a new PR#267.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 10, 2026
@FileSystemGuy FileSystemGuy deleted the FileSystemGuy-newRules branch March 10, 2026 22:01
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