-
Notifications
You must be signed in to change notification settings - Fork 422
Add vtr_reg_weekly_no_he that drops the high effort Titan tests and gaussian blur #1118
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
Conversation
…blur Signed-off-by: Keith Rothman <[email protected]>
Signed-off-by: Keith Rothman <[email protected]>
@kmurray Dropping the high effort titan tests (and gaussian blur) and running with 3 cores means that the "weekly" test set returns in ~40 hrs, which isn't bad! I've added a CPU and memory load log every 5 minutes in this PR, which will hopefully identify why the CI takes "forever" if a large memory regression is introduced. |
@kmurray : Is there a better way to compose the |
I've considered that, and it's potentially possible but would need some scripting work. How I'd approach it would be to extend the
This would support selectively not running the HE titan test by using the appropriate task list. Selectively dropping a particular benchmark (e.g. gaussianblur) would be more difficult. |
Once we have the include feature, it is straightforward to make 3 task lists. Do you need the deduplication effort solved right now, or is this PR acceptable? |
Discussed in meeting. Don't need this PR, and instead just stop doing weekly as a pre-submit (do as post-merge only). |
Description
This PR is attempting to create a weekly that can provide the vast majority of the signal, without the excessive runtime.
Related Issue
Motivation and Context
This PR is attempting to do two things:
How Has This Been Tested?
Types of changes
Checklist: