search

mlcommons / training_policies

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

Rules state that 1000 or 1001 classes are allowed for RN50 in 2 places #329

Closed jonathan-cohen-nvidia closed 4 years ago

jonathan-cohen-nvidia commented 4 years ago

Minor point, but this is now in 2 places: https://github.com/mlperf/training_policies/blob/master/training_rules.adoc#94-equivalence-exceptions and https://github.com/mlperf/training_policies/blob/master/training_rules.adoc#12-appendix-benchmark-specific-rules

We should delete one of them. @petermattson

petermattson commented 4 years ago

Good catch. https://github.com/mlperf/training_policies/pull/333

On Thu, Jun 4, 2020 at 6:39 PM jonathan-cohen-nvidia < notifications@github.com> wrote:

Minor point, but this is now in 2 places:

https://github.com/mlperf/training_policies/blob/master/training_rules.adoc#94-equivalence-exceptions and

https://github.com/mlperf/training_policies/blob/master/training_rules.adoc#12-appendix-benchmark-specific-rules

We should delete one of them. @petermattson https://github.com/petermattson

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mlperf/training_policies/issues/329, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIIVUHL6X2DXL3H6PYYCUJDRVBEDJANCNFSM4NTEDJYA .

petermattson commented 4 years ago

Promoting this to rule change so group can confirm PR, since no actual content change.

petermattson commented 4 years ago

This has been fixed by a merged PR.