-
Notifications
You must be signed in to change notification settings - Fork 35
Test Enzyme on demo models #813
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
Conversation
25b6c7e
to
a145999
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #813 +/- ##
==========================================
- Coverage 84.43% 4.31% -80.13%
==========================================
Files 34 34
Lines 3849 3804 -45
==========================================
- Hits 3250 164 -3086
- Misses 599 3640 +3041 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
a145999
to
cdbde76
Compare
f42a6e1
to
409fc3b
Compare
8f8018a
to
566257e
Compare
Pull Request Test Coverage Report for Build 13958820972Details
💛 - Coveralls |
Benchmark Report for Commit
|
Superseded by ADTests, I think, since that now contains all the demo models. |
This PR is mostly a speculative attempt to see how Enzyme fares on our demo models.
Enzyme AD takes very long to run (technically, the preparation step takes a long time) and so this PR comments out all other tests (including AD tests on other backends).
tldr so far: All tests pass (both forward- and reverse-mode of Enzyme) with all demo models when using TypedVarInfo, but UntypedVarInfo segfaults