Skip to content

Trying jax for gradients - #24

Open
jmlarson1 wants to merge 20 commits into
mainfrom
feature/jax_for_gradients
Open

Trying jax for gradients#24
jmlarson1 wants to merge 20 commits into
mainfrom
feature/jax_for_gradients

Conversation

@jmlarson1

@jmlarson1 jmlarson1 commented Sep 30, 2025

Copy link
Copy Markdown
Collaborator

Our python gradients/jacobians were computed in matlab by AdiMAT and then translated to python.

We can also compute them using Jax. This allows us to easily compute Hessians (if desired).

@jmlarson1
jmlarson1 marked this pull request as ready for review October 29, 2025 18:18
@jmlarson1
jmlarson1 requested a review from mmenickelly October 29, 2025 18:20

@mmenickelly mmenickelly left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I appreciate the test, it runs locally and confirms that there is no significant difference in the Jax and numpy output. I would leave that test in here for now in case we suspect weird things on a new environment.

Otherwise, this is all a good idea in principle and it works for this limited dataset, so it's an acceptable PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants