-
-
Notifications
You must be signed in to change notification settings - Fork 116
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
DEDataArray factorization overloads cause type ambiguity errors #648
Comments
This is starting to go into bad news bears territory because linear algebra doesn't always respect the propagation of the non-array variables, so it might have different semantics than you want. This is actually why I think I'm going to deprecate DEDataArray. |
You mean because of the distinction between
|
I mean it's undefined how to copy things through some operations. I just don't think that is good behavior to have. And it carries memory in ways that are not optimal. I think MTK is just going to be more efficient and have more clean semantics in all but a very few cases. |
But doesn't MTK turn everything into an ODEProblem in the end? Seems like you would still have this problem. I experimented with MTK early on and it seemed like it wasn't really able to do this (i.e. have non-integrated state). |
It now has a whole observables system. MTK is a very fast moving project. |
But can an integrated variable then depend on an observable? |
Yes. You can't effect it in an |
I'm not sure if this is intended or not, but it appears that the existing
DEDataArray
factorization overloads cause type ambiguity errors for stiff solvers, e.g:causes a type ambiguity error with the the
LU
factorization methods inLinearAlgebra
becauseDEDataArray
is more specific thanAbstractArray
butFactorization
is less specific thanLU
.This forces the user to define their own more specific method to resolve the ambiguity, which kind of seems to nullify the point of providing this method in the first place...?
Shouldn't these functions "just work" without these ambiguous overloads since
DEDataArray
is already anAbstractArray
?The text was updated successfully, but these errors were encountered: