-
Notifications
You must be signed in to change notification settings - Fork 49
Quick tune strix navi48 #2014
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
Quick tune strix navi48 #2014
Conversation
…snet model configs" This reverts commit 171b6a2.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please can you use handleNewConfigs.py to update configs/tier1-*-configs ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. What does handleNewConfigs.py do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it splits the file into the files we actually use for tuning. Not sure if we want to do this here or a next PR? @dorde-antic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would make sense to do this in a separate PR since it affects the older configs as well. Having one clean PR that implements this change is going to be better for history/tracking purposes.
Motivation
Technical Details
Test Plan
Test Result
Submission Checklist