Skip to content

Added more fields to the schema of mlperf training summary results#455

Draft
nnasirinvidia wants to merge 2 commits intomlcommons:masterfrom
nnasirinvidia:logging_schema_changes
Draft

Added more fields to the schema of mlperf training summary results#455
nnasirinvidia wants to merge 2 commits intomlcommons:masterfrom
nnasirinvidia:logging_schema_changes

Conversation

@nnasirinvidia
Copy link
Copy Markdown

Added GBS, RCP, Energy, Samples to Converge and RCP Scaling Factor to the result summarizer.

@nnasirinvidia nnasirinvidia requested review from a team as code owners April 28, 2026 21:31
@github-actions
Copy link
Copy Markdown

MLCommons CLA bot:
Thank you very much for your submission; we really appreciate it. Before we can accept your contribution,
we ask that you sign the MLCommons CLA (Apache 2). Please submit your GitHub ID to our onboarding form to initiate
authorization. If you are from a MLCommons member organization, we will request that you be added to the CLA.
If you are not from a member organization, we will email you a CLA to sign. For any questions, please contact
support@mlcommons.org.
0 out of 1 committers have signed the MLCommons CLA.
@nnasirinvidia
You can retrigger this bot by commenting recheck in this Pull Request

Comment thread mlperf_logging/result_summarizer/result_summarizer.py
Comment thread mlperf_logging/result_summarizer/result_summarizer.py
Comment thread mlperf_logging/result_summarizer/result_summarizer.py
@ShriyaRishab
Copy link
Copy Markdown
Contributor

ShriyaRishab commented Apr 28, 2026

Thanks for creating this, Nasibeh! It needs some cleanup but looks good.

@pgmpablo157321 @mrmhodak here is a draft PR that adds additional fields to the result summarizer csv

Can you please review it?

@ShriyaRishab ShriyaRishab marked this pull request as draft April 28, 2026 21:37
@mrmhodak
Copy link
Copy Markdown

Seems to be doing the right thing.

Can see the output this generates on 5.1 results and review it during the WG meeting?

@nnasirinvidia
Copy link
Copy Markdown
Author

@ShriyaRishab Can you please share the CLA form that I can sign it?

@ShriyaRishab
Copy link
Copy Markdown
Contributor

@nnasirinvidia this message has the CLA instructions - #455 (comment)

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.

3 participants