-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Limiting tests: reducing the time of the news recommendation GPU notebooks #1656
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
@pradnyeshjoshi, in a VM could you please run:
There are two errors that might be expected:
|
|
@pradnyeshjoshi the times between your VM and the test VM vary a lot, even though the @anargyri @laserprec there is a weird error in the coverage, any idea of what could be the problem? |
@miguelgfierro I'm using Standard NV6 (6 vcpus, 56 GiB memory). |
Not sure, but it has to do with switching to an AzureML VM for the tests. |
@pradnyeshjoshi do you think we can merge this PR? |
The error is relating to merging all the code coverage reports from the various runs before sending to codecov. When merging, As it turns out, codecov can merge the reports for us, I think we can try skipping the merging step and send all of the |
Description
Reducing the test time of the limiting tests. I reduced the epochs and increased the batch size
Related Issues
related to #995 (comment)
Checklist:
staging branch
and not tomain branch
.