mlcommons / training_policies

Issues related to MLPerf™ training policies, including rules and suggested changes
https://mlcommons.org/en/groups/training
Apache License 2.0
92 stars 66 forks source link

Fix some references to Training rules doc #464

Closed sparticlesteve closed 2 years ago

sparticlesteve commented 3 years ago

Using section links rather than referring to rule numbers.

Fixes #462

github-actions[bot] commented 3 years ago

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

sparticlesteve commented 3 years ago

@johntran-nv this is basically a docs bugfix/cleanup. Can you take a look?

TheKanter commented 2 years ago

Hi @bitfort and @johntran-nv can we put at top of queue for Thursday meeting?

johntran-nv commented 2 years ago

Since this is only touching the hpc rules doc, I'd propose that we can merge this as long as the HPC WG approves, and we don't need Training WG approval. Does that sound reasonable, @TheKanter ?

If so, I can't tell if the HPC WG has discussed this one yet - have they?

sparticlesteve commented 2 years ago

It's not a rules change. It just changes the way our hyperlinks to Training rules are implemented. It shouldn't require any discussion.

johntran-nv commented 2 years ago

Good point. I agree that it's ok to merge without discussion. Merging now.