Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add a column of self-reported precision/numerics to MLPerf results table #190

Open
christ1ne opened this issue Dec 14, 2020 · 6 comments
Open
Labels

Comments

@christ1ne
Copy link
Contributor

Currently there is no easy way for the table users to tell whether it is a FP32, INT8 or other numerical being used.
This essentially discourages any FP32 submission but most inference are still in FP32. Adding the numerics will help the users to further determine the relevance of the results to their needs.

@tjablin @DilipSequeira FYI.

@christ1ne christ1ne changed the title add a column of self-reported precision/numerics to MLPerf results add a column of self-reported precision/numerics to MLPerf results table Dec 14, 2020
@christ1ne
Copy link
Contributor Author

WG:

  • concerns: self-reporting will lead to messy outcome
  • objective should be define the efforts required to produce this result
  • please add your thoughts offline to this issue. we will discuss again in Jan, 2021.

@christ1ne
Copy link
Contributor Author

WG: need a specific example AR: @christ1ne

@christ1ne
Copy link
Contributor Author

christ1ne commented Jan 25, 2021

precision_example.xlsx
@DilipSequeira please let me know what you think? This precision info is from the json in the measurement directories.

@DilipSequeira
Copy link
Contributor

The format looks reasonable (I would slightly prefer colors to postfix characters but I expect there are UI accessibility concerns.)
A larger problem is having rules for transforming from what is essentially a freeform text field (a submitter could reasonably write there e.g. "INT8/BF16" if they're using mixed-precision) to a precision annotation. How would we handle that case?

@christ1ne
Copy link
Contributor Author

We allow all numerics but we should have a good list to take care of different capitalization, naming nuances.

@christ1ne
Copy link
Contributor Author

WG:
concerns:

  1. symbols are hard to work with in a numerics table.
  2. The list of precision choices could be hard to determine. -> we can defer the list of precision choices to the review period

However the WG think we should do this and we will find a UI solution in the review period. Possibly a different view of the results table in a different link.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants