Skip to content
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

adjoint and algebra solver #431

Closed
wants to merge 9 commits into from
Closed

adjoint and algebra solver #431

wants to merge 9 commits into from

Conversation

NicholasCowie
Copy link
Contributor

@NicholasCowie NicholasCowie commented Aug 4, 2023

Summary

We are in need of faster solving and simultaneously better steady state checking if the algebra solver is used.
I’ve included additional unit tests so we can be sure that each of the configurations are solving to the correct steady state.

The basic premise is to solve the ODE solution for a period of time so that it is close to the steady state solution and will be numerically stable when solving. Additionally, we’ve included the adjoint solver which produces more efficient solving -- even when using the algebra solver. This indicates that the backwards solution to the ODE is still being calculated despite being not used in the log-probability calculation.

Reason

  • faster ode solving.

Drawbacks

  • more parameters and finer tuning to ensure stable sampling.

Checklist:

  • Updated any relevant documentation
  • Add an adr doc if appropriate
  • Include links to any relevant issues in the description
  • Unit tests passing
  • Integration tests passing

@NicholasCowie NicholasCowie added the Work In Progress Don't merge yet! label Aug 6, 2023
@NicholasCowie NicholasCowie linked an issue Aug 6, 2023 that may be closed by this pull request
@NicholasCowie NicholasCowie removed the Work In Progress Don't merge yet! label Aug 6, 2023
@NicholasCowie NicholasCowie changed the title [WIP] adjoint and algebra solver adjoint and algebra solver Aug 6, 2023
Copy link
Member

Choose a reason for hiding this comment

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

I think you can avoid the line too long error here by just renaming config to mc

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.

Adjoint Algebraic hybrid
2 participants