-
Notifications
You must be signed in to change notification settings - Fork 122
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
Add support for initializer_list in forward mode AD #935
Conversation
clang-tidy review says "All clean, LGTM! 👍" |
10abd21
to
75c15d0
Compare
clang-tidy review says "All clean, LGTM! 👍" |
75c15d0
to
0cab862
Compare
clang-tidy review says "All clean, LGTM! 👍" |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #935 +/- ##
=======================================
Coverage 93.86% 93.86%
=======================================
Files 55 55
Lines 7892 7894 +2
=======================================
+ Hits 7408 7410 +2
Misses 484 484
|
0cab862
to
5710a97
Compare
clang-tidy review says "All clean, LGTM! 👍" |
5710a97
to
00d4c29
Compare
clang-tidy review says "All clean, LGTM! 👍" |
00d4c29
to
d62f3d7
Compare
clang-tidy review says "All clean, LGTM! 👍" |
b179e19
to
5ca26d0
Compare
clang-tidy review says "All clean, LGTM! 👍" |
5ca26d0
to
90782dc
Compare
clang-tidy review says "All clean, LGTM! 👍" |
This commit adds primitive support for initializer_list in the forward mode AD.
90782dc
to
20d08d4
Compare
clang-tidy review says "All clean, LGTM! 👍" |
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!
This commit adds primitive support for initializer_list in the forward mode AD.