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
Eager Torch #438
Eager Torch #438
Conversation
Check out this pull request on You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB. |
@analysiscenter/batchflow currently, main subject of the review should be Current TODO: |
Codecov Report
@@ Coverage Diff @@
## master #438 +/- ##
=========================================
Coverage ? 35.92%
=========================================
Files ? 149
Lines ? 14459
Branches ? 0
=========================================
Hits ? 5195
Misses ? 9264
Partials ? 0
Continue to review full report at Codecov.
|
Implemented #366 Something weird is happening with |
This PR proposes to add
EagerTorch
model that:can build off of
batch_data
during first call of thetrain
methodallows for better usage of native
torch
modulesdoes not use redundant
tf
-like methods (make_inputs
,has_classes
, etc)