-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[AutoDiff] Refactor AutoDiff system and support for loops that carry mutable states #1285
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1285 +/- ##
==========================================
- Coverage 85.09% 84.72% -0.37%
==========================================
Files 18 18
Lines 3273 3274 +1
Branches 616 616
==========================================
- Hits 2785 2774 -11
- Misses 352 365 +13
+ Partials 136 135 -1
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.
LGTM! Just a few nits.
Co-authored-by: xumingkuan <[email protected]>
Related issue = #581 #1274
I confirmed that all the difftaichi demos works now.
@xumingkuan a high-level design document of the new autodiff system (which I cannot make public now) is emailed to you. Hope that will make reviewing easier.
[Click here for the format server]