-
Notifications
You must be signed in to change notification settings - Fork 14
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
#79 dramatically worsened simulation times and causes large simulation errors #80
Comments
The 50 s timeout likely plays some role, but we still get successful simulation and verification, so it's not failing. Not sure what is going on. |
I remember you said that if you have and output-interval as non integer that it would get stuck, maybe this is what it happens and the timeout just forces to finish (correctly). |
Yes, that could be, see CATIA-Systems/FMPy#655. But then how comes we still get the verification right? Could it be that it is still using the previously computed result file? If so, then you should probably update the test script to delete all such file before starting the new simulation. |
We always delete the previous results so that's not the case. |
OK. Any clue what is actually going on there? |
@adrpo, please check https://libraries.openmodelica.org/branches/history/master-fmi-fmpy/2024-03-18%2022:22:01..2024-03-19%2020:08:50.html. Apparently, many simulations went from a few seconds to fifty-something.
I'm not sure if this is just a glitch in reporting, these timings are a bit suspicious. How can we check?
The text was updated successfully, but these errors were encountered: