You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
!(symtype(diffvar) === Real ||eltype(symtype(diffvar)) === Real) &&
throws an error for previously working examples / tests in QuantumCumulants.jl.
Tests error with, e.g.
ArgumentError: Differential variable var"⟨a′*a⟩"(t) has type Complex. Differential variables should not be concretely typed.
The check is too restrictive IMHO, since it's not strictly necessary for things to be <: Real here. Would it be possible to loosen the constraints? Would it be enough to check whether the type is abstract and <: Number?
Also, if you want to be picky: the check doesn't actually do what the error message says. Checking for symtype(x) === Real isn't actually checking for concretely typed variables.
The text was updated successfully, but these errors were encountered:
The recently introduced type check here
ModelingToolkit.jl/src/utils.jl
Line 1240 in 57c79e9
Tests error with, e.g.
The check is too restrictive IMHO, since it's not strictly necessary for things to be
<: Real
here. Would it be possible to loosen the constraints? Would it be enough to check whether the type is abstract and<: Number
?Also, if you want to be picky: the check doesn't actually do what the error message says. Checking for
symtype(x) === Real
isn't actually checking for concretely typed variables.The text was updated successfully, but these errors were encountered: