Skip to content

Additional Disclosure - #582

Closed
mrmhodak wants to merge 1 commit into
mlcommons:masterfrom
mrmhodak:patch-2
Closed

mrmhodak wants to merge 1 commit into
mlcommons:masterfrom
mrmhodak:patch-2

Conversation

@mrmhodak

@mrmhodak mrmhodak commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Give additional disclosure with the results to help public understand what was done

Give additional disclosure with the results to help public understand what was done
@mrmhodak
mrmhodak requested review from a team as code owners April 9, 2026 15:36
@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

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

Comment thread training_rules.adoc

Results need to be informative and provide basic information about how they were run and convergence without needing to run additional scripts or consulting conmfigurational files. Specifically, the results the following must be disclosed as a part of each result

* Smallest (most compact) numerical precision used to accelerate training

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Since that does not paint a real picture of the results - shouldn't we require more information about the numerical precision recipe used for training? Like which inputs, activations, weights, gradients, optimized states use which precision; and what techniques are used to stabilize convergence?

Comment thread training_rules.adoc

* Smallest (most compact) numerical precision used to accelerate training

* Parallelization techniques employed, such as CP8, PP4, or TP1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Documenting this information is error prone and adds additional overhead for submitters. Can we make these MLLOG tags so the results file shows this information?

Comment thread training_rules.adoc

* RCP checker output

Additionally, configuration files must be named so that they can easily be matched with output files.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Or, the submitter needs to ensure that there's a clear way to figure out which config file was used for which submission.

@ShriyaRishab

Copy link
Copy Markdown
Contributor

Added directly into the logging repo

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants