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

Require logging of seeds for closed submissions #546

Open ShriyaPalsamudram opened 2 months ago

ShriyaPalsamudram commented 2 months ago

Currently, the seed checker within the package checker ensures that logged seeds are compliant. But it only raises a warning if seeds are not logged as stated here.

But we do not have a rule enforcing that all CLOSED submissions need to log the seed used although if seeds are logged, the package checker ensures that unique seeds are used in each run.

Since we expect all submissions to be run with different seeds and that the logged seed is proof that not all submission log files are run with the same seed, we need to make logging of seeds as a requirement instead of keeping it as an optional parameter.

(Creating this issues based on Training WG discussion on Sept 12th, 2024)