-
Notifications
You must be signed in to change notification settings - Fork 31
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
Jax export #1861
Jax export #1861
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #1861 +/- ##
===========================================
- Coverage 77.83% 77.65% -0.19%
===========================================
Files 325 329 +4
Lines 21853 22054 +201
Branches 1456 1456
===========================================
+ Hits 17009 17125 +116
- Misses 4833 4918 +85
Partials 11 11
Flags with carried forward coverage won't be shown. Click here to find out more.
|
SonarCloud Quality Gate failed. |
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
|
Would be great to get some initial feedback. Quite some room for later optimization, but seems functional now. If API and overall code looks good I would also add a short tutorial notebook to the documentation. |
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.
Thanks, looks good to me. I am not familiar with the jax ecosystem, though.
Having a tutorial/example soon would be great, yes.
The new dependencies bring in a couple of valgrind issues that need to be added to the suppression list: https://github.com/AMICI-dev/AMICI/actions/runs/11888412281/job/33122901845 |
Quality Gate passedIssues Measures |
No description provided.