Skip to content

Conversation

@roystgnr
Copy link
Contributor

This requires the newest MetaPhysicL (not the newest release, the
stuff I pushed to github 60 seconds ago) to pass. The test will be
skipped if Antioch is configured without MetaPhysicL.

@hovr2pi, this is the basics of what we want for getting massive parameter sensitivity gradients out of Antioch calls. The remaining catches are going to be application-side: indexing (we'll need some kind of factory method for associating unique integers to each parameter we try to differentiate against) and setting the independent variable derivatives (this is easy for kinetics parameters, thanks to #135, but Varis was going to add the equivalent access for transport and thermo parameters but IIRC never found time).

This requires the newest MetaPhysicL (not the newest release, the
stuff I pushed to github 60 seconds ago) to pass.  The test will be
skipped if Antioch is configured without MetaPhysicL.
@pbauman
Copy link
Member

pbauman commented Apr 14, 2016

@roystgnr So is this an AD solution to achieve similar functionality as #142? (We should try and get that merged "soon".)

@roystgnr
Copy link
Contributor Author

It is. The big hope is that we can thus more easily get gradients w.r.t. lots and lots of reactions without lots and lots of code (in this case, 0 lines in library vs 2000).

@pbauman
Copy link
Member

pbauman commented Apr 14, 2016

I think it will be good to have both as an option, if for no other reason then things like timings etc.

At any rate this is awesome!

@roystgnr roystgnr merged commit 8e658fa into libantioch:master Apr 14, 2016
@roystgnr roystgnr deleted the multiderivative_test branch April 14, 2016 18:14
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