-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
AzureML Tests #1717
AzureML Tests #1717
Conversation
Codecov Report
@@ Coverage Diff @@
## staging #1717 +/- ##
============================================
+ Coverage 0.00% 23.20% +23.20%
============================================
Files 87 88 +1
Lines 9132 9142 +10
============================================
+ Hits 0 2121 +2121
- Misses 0 7021 +7021
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
Adding some comments
One question, for these tests to be enabled, do you need them to be in main?
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 great @pradnyeshjoshi,
Did you get the final time we have now?
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.
Great work @pradnyeshjoshi ! ❤
@miguelgfierro time taken for last runs: unit tests - 28 minutes, nightly - 42 minutes |
merge with latest staging
merge with latest staging
…rs into pradjoshi/aml_tests
…rs into pradjoshi/aml_tests
…rs into pradjoshi/aml_tests
For reference, the latest release took 9h 49m (see run), with this system, all the tests will take 42min. OH YEAH!!! big congrats @pradnyeshjoshi! |
Description
Main contributions of this PR:
Improvements:
Related Issues
Feature request: #995
Checklist:
staging branch
and not tomain branch
.