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

Clean-up steady-state solver code #1073

Open
7 of 9 tasks
paulstapor opened this issue May 6, 2020 · 4 comments
Open
7 of 9 tasks

Clean-up steady-state solver code #1073

paulstapor opened this issue May 6, 2020 · 4 comments

Comments

@paulstapor
Copy link
Contributor

paulstapor commented May 6, 2020

Currently, the workSteadyStateProblem routine isn't exactly the nicest part of Amici...
Things to be changed here in some close future:

@paulstapor paulstapor self-assigned this May 6, 2020
@paulstapor paulstapor added the c++ label May 6, 2020
@dweindl
Copy link
Member

dweindl commented May 16, 2020

* [ ]  Remove the triple try-catch nesting, when determining whether or not to use simulation

Very much in favor. This is a huge pain to debug.

@FFroehlich
Copy link
Member

  • correct the (partly) erroneous or misleading output into rdata of the steady state solver

I have no idea what this refers to, anyone?

@dweindl
Copy link
Member

dweindl commented Mar 23, 2022

No idea.

@dweindl
Copy link
Member

dweindl commented Mar 23, 2022

* [ ]  create figure for documenting the steady state solver code via sphinx, rather than having something prerendered

Started here. Might need some further updates already.
These diagrams can be included in sphinx via https:/mgaitan/sphinxcontrib-mermaid.

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

No branches or pull requests

4 participants