-
Notifications
You must be signed in to change notification settings - Fork 7
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
Handle TFM_INS
variants
#120
Comments
Thanks @siddharth-krishna. the |
Probably the easiest way to test this is to modify an existing benchmarks. Should I do it? |
We should also add |
Modifying one of the demos sounds good! Will be easier to debug :) |
Ok, I've changed the fifth one... |
No, I believe |
Thanks @Antti-L. I've overlooked the typo. @siddharth-krishna please notice the difference between |
BTW, I hope you are aware that |
Thanks @Antti-L. We should open a separate issue on |
These variants essentially allow specifying multiple attribute values in a row by un-melting a field (e.g. attribute, year) and allowing them to be column names. So to handle them, we find those columns, and melt it back into 2 columns: attribute & value (or year & value).
TFM_INS-TS
:Year
TFM_INS-AT
:Attribute
times-info.json
(everything with category =parameter
)TFM_INS-AT
in any of our benchmarks. What should we use to test this?TFM_INS-TXT
:@olejandro I'm afraid I'll miss the meeting today, my update is that I'm working on this feature, and need some guidance with the questions above. I've implemented
TFM_INS-TS
but now I'm fixing some regression bugs.The text was updated successfully, but these errors were encountered: