Martin J. Kühn

@sudiphzi @MariamaJ This merge request does not address the issue at all. It is rather reformatting of the original code. Please create corresponding issues and assign pull requests correctly. Please...

@nijawa : Yes, nan is probably an incorrect input but clamp should prevent any appearance of nan **beforehand**. So there's potentially another problem? My questions still is "How does the...

I think that `compartment / step_size * step_size` can differ from `compartment` is not the problem, that basically is the thing for all flops. The problem, as you state, rather...

Yes, the change would just address the functionality not merge the model yet. As we are in double precision, `eps` (or I propose `tol`) should be much smaller. Minimum 1e-10...

@reneSchm I think this is a good idea! Maybe we should only check if Vector is not something that hides another structure behind and take a naming that is less...

No let's go with it then.

@reneSchm I think there are some things like Eigen structures that always explicitly use double while our custom stuff could be built with float or double for instance. This could...

@lenaploetzke Can you check if #983 resolves the pygeneration issue here? @reneSchm Can you merge the new main and check that the integrator gets resolved correctly?

@MaxBetzDLR @reneSchm I cannot merge this. The CI is still failing. Can you address this tomorrow?

@hannemann-tamas Can you check the boxes in the pull request above and provide a bullet point list with the changes you did?