-
Notifications
You must be signed in to change notification settings - Fork 5
Additional Enzyme examples and use cases (e.g DistributedGenerator) #54
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
Co-authored-by: pelesh <peles2@llnl.gov>
…ed to find a better way.
Perhaps we should consider breaking this into a series of PRs, each with very specific focus? |
I agree. It's getting harder to keep this branch up to date with develop as well. |
I marked the PR as ready for review. I will create other PRs for the next features. |
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.
Left mentions on some variable names that should be changed throughout.
There are no comments on any of the test functions.
Code passes all tests, so I approved it.
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.
Overall looks good to me.
…for a follow-up PR.
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.
Looks good. Some code style issues still need to be addressed.
) * Add preliminary Enzyme support * Use better naming conventions * More verbose tests for enzyme support. * int --> double in Enzyme examples. * Separate scalar and vector examples using Enzyme. * Use classes in EnzymeLibCheck. * Basic usage of DenseMatrix in EnzymeVector example. * Method to convert Dense matriz to COO. * Use DenseMatrix to store the Jacobians in Enzyme vector example. * Working Enzyme derivative of DistributedGenerator * EnzymeLibVectorCheck. * Better documentation of the DenseMatrix class. --------- Co-authored-by: Asher Mancinelli <ashermancinelli@gmail.com> Co-authored-by: pelesh <peles2@llnl.gov> Co-authored-by: Slaven Peles <peless@ornl.gov>
This is work in progress to use Enzyme to compute Jacobians
Status: