-
Notifications
You must be signed in to change notification settings - Fork 88
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 benchmark CI job for fast checks #1300
Conversation
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.
Looks good to me! But given we change CI I would also wait for @gpmayorga approval
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.
LGTM 🌞
This PR fails as expected due to the Once that PR is merged, this can be safety merged. |
I think we can run better the three runtimes in the same job to save 1 hour per CI job |
3b86366
to
d8c213c
Compare
d8c213c
to
887b0a7
Compare
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.
LGTM!
Description
This job does not generate weights, it only checks fastly for all runtimes if benchmarks work/compile.
You can test the script locally as follows for fast benchmark checks (it takes around 10-15s to check all runtime benchmarks once it's compiled):
Being
<runtime>
the following possible values:development
,altair
,centrifuge
Fixes #1268
Changes and Descriptions