Venkateshprasad
Venkateshprasad
The newly added tests in "Variable Utils Tests" have passed [here](https://github.com/SciML/ModelingToolkit.jl/actions/runs/8911493073/job/24472967508?pr=2583#step:6:1390) As `Clock` calls `default_toterm` and the behavior shouldn't change "Parameter Dependency Test" are related too. It has passed [here](https://github.com/SciML/ModelingToolkit.jl/actions/runs/8911493073/job/24472967508?pr=2583#step:6:1398)...
Vanilla `@variables` doesn't support that; I'll see how that can be added. But note that inside the `@mtkmodel`, such a thing just works.
Error is because the `r` wasn't passed to `ODESystem`. ```julia @component function Temp(; name, r) : ODESystem(eqs, t, [], [r...]; name) end ``` Maybe this[^1] error message can be changed...
New tests are added to "Model Parsing" and they pass [here](https://github.com/SciML/ModelingToolkit.jl/actions/runs/8832467051/job/24249893731?pr=2501#step:6:1301). Test failures are unrelated to this PR. Documentation had passed for the first commit [here](https://github.com/SciML/ModelingToolkit.jl/actions/runs/8830257057/job/24242990387?pr=2501). The current failure is...
This PR is ready for review.
There is an extra comma after the equation. So it ends up being `D(A) ~ (((1 / VA) * (QCSF * (c_in - A) - Qosc * (A - B)),...
#214 is switching to use DynamicQuantities. We need ability to define custom units (like Wb). The foll. PR adds that: - https://github.com/SymbolicML/DynamicQuantities.jl/pull/107
Related: - https://github.com/JuliaSymbolics/Symbolics.jl/pull/1104 - https://github.com/SciML/ModelingToolkit.jl/pull/2583
Looks like `--threads` wasn't allowed in Julia 1.0.5. ~I suppose, we would want to avoid a version based statement. So, instead of this, setting `JULIA_NUM_THREADS` in `env` is the best...
This is no longer required.