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

Check how rk45 handles assignment rules #445

Open
kirichoi opened this issue Feb 13, 2018 · 5 comments
Open

Check how rk45 handles assignment rules #445

kirichoi opened this issue Feb 13, 2018 · 5 comments
Assignees

Comments

@kirichoi
Copy link

No description provided.

@evilnose
Copy link

Hi Kiri, may I close this?

@kirichoi
Copy link
Author

I think rk45 solver cannot handle assignment rules properly. I don't think this is a priority anymore but should be investigated sometime.

@kirichoi kirichoi removed the priority label May 26, 2020
@matthiaskoenig
Copy link
Collaborator

If there is an issue it should be picked up by the SBML test suite. I.e. in the continious integration testing the SBML TestSuite should be run with the various integrators, i.e.

  • run all deterministic tests with all deterministic integrators
  • run all stochastic tests with all stochastic integrators

@hsauro
Copy link

hsauro commented May 27, 2020 via email

@luciansmith
Copy link

I checked the rk45 and rk4 integrators against the test suite. The rk45 integrator ran until test 65, and then got into an infinite loop, after failing nearly 100% of the tests before that. I then changed 'epsilon' to 1e-10, which I had found worked for test 1, and re-ran; it then got to test 172, (passing maybe 10 of them, likely the 'soft' tests), at which point it crashed when trying to calculate the factorial of 1315018.

The rk4 integrator did somewhat better--it never crashed/stopped, and successfully passed 1214 out of 1632 of the tests, which is only 74%, but it's a lot higher than 1%...

Given this, it looks like running those integrators against the test suite in general is a losing proposition. We do some spot checking of the integrators in the .rrtest files in Python, so they're at least checked to make sure they run, but more extensive testing seems out of our grasp at the moment.

However, I did implement #853, which should allow us to test integrators against the test suite more easily in the future.

Leaving this issue open for now; it would indeed still be nice to test these integrators.

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

5 participants