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
Percentage FLOPS or Multiply Adds - inspired by #199 #200
Comments
This seems like a good idea, PRs appreciated :) I would prefer some cleaner way of providing the percentage form of all of these measurements though. If we keep going we'll have 2x the options for all columns (num_params, percent_params), (num_macs, percent_macs), etc. But if we can't think of anything better, adding a few extra params would be fine. |
I agree that there should be prettier API to deal with these.
Just brainstorming, some of them may seem unnecessarily complex:) |
Sorry, maybe it was not clear. Do you any feedback on the list of formats I suggested? I will move on based on a particular choice of format. |
I guess 1) is okay for now. From the options, I think anything else will make the API more confusing |
Option for column representing Percentage FLOPS or Multiply Adds. (similar to #199)
I think this option would be really useful to see which part of model should be optimized if necessary. It also useful to have an idea about the scaling of the model as make your model bigger and bigger.
At first sight, it seems that this could be implemented in a way similar to that of #199.
If this seems reasonable enough, I can come up with PR.
The text was updated successfully, but these errors were encountered: