Closed elswit closed 4 years ago
I think all the issues have been addressed. I updated Problem
and Rhs
to support more odeset
options and renamed the MassMatrix
property of Rhs
to Mass
for consistency with odeset
.
Does this address the InitialSlope issue? I know you did some MassSingular stuff.
Yeah, InitialSlope
isn't necessary if MStateDependence
is set.
The only problem I see then is operator spacing. In most of the code we have
-1a + 2b + 3*c
instead of
-1 a + 2 b + 3 * c
like the code in this branch now has.
should we prefer one over the other?
I'm not that bothered about the multiplication spacing. We would have to change many files to make that consistent.
OK. I tend to agree. I don't think there are any more issues then, correct?
This DAE is interesting bc it's the only one in OTP with time-dependent mass matrix.