Skip to content
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

Fixes requirements to be installable, fixes CUDA Tests #88

Merged
merged 4 commits into from
Dec 11, 2020
Merged

Fixes requirements to be installable, fixes CUDA Tests #88

merged 4 commits into from
Dec 11, 2020

Conversation

oke-aditya
Copy link
Owner

@oke-aditya oke-aditya commented Dec 11, 2020

Our last step to #75
After this will release rc2

I have manually shifted model to CPU and we will do this for all CPU operations.
Sometimes while running GPU tests, it causes device mismatch and we can avoid that.

@oke-aditya oke-aditya self-assigned this Dec 11, 2020
@oke-aditya oke-aditya added the release For Releasing label Dec 11, 2020
@codecov
Copy link

codecov bot commented Dec 11, 2020

Codecov Report

Merging #88 (8c08c80) into master (d098527) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #88   +/-   ##
=======================================
  Coverage   78.05%   78.05%           
=======================================
  Files          49       49           
  Lines        1581     1581           
=======================================
  Hits         1234     1234           
  Misses        347      347           
Flag Coverage Δ
unittests 78.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d098527...8c08c80. Read the comment docs.

@oke-aditya oke-aditya merged commit 395b205 into oke-aditya:master Dec 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release For Releasing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant