search

mlcommons / inference_policies

Issues related to MLPerf™ Inference policies, including rules and suggested changes
https://mlcommons.org/en/groups/inference/
Apache License 2.0
55 stars 52 forks source link

Add audit guidelines #258

Closed nvzhihanj closed 1 year ago

github-actions[bot] commented 1 year ago

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

rnaidu02 commented 1 year ago

@tjablin @psyhtest Can you review and provide your comments by next week?

tjablin commented 1 year ago

This seems very reasonable to me. Would you consider adding some text to make it clear that the auditor has the discretion to examine anything they find suspicious or interesting beyond the enumerated points?

nvzhihanj commented 1 year ago

This seems very reasonable to me. Would you consider adding some text to make it clear that the auditor has the discretion to examine anything they find suspicious or interesting beyond the enumerated points?

Added. Thanks for the suggestion!

nv-ananjappa commented 1 year ago

@rnaidu02 Could you merge since all review comments have been addressed?

nvzhihanj commented 1 year ago

@rnaidu02 kindly checking again for the review/approval, thanks.

nv-ananjappa commented 1 year ago

@tjablin Could you please review and see if any other changes are required?