Fix @mtkmodel independent variable generation #3001
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This should fix https://discourse.julialang.org/t/how-do-i-import-t-nounits-with-a-different-name-than-t-in-modelingtoolkit/118640.
The issue was that the
@mtkmodel
macro internally generates independent variables as "variables", and not "parameters" (which@independent_variables
does). I have tried to fix this in a way which is consistent with the rest of the code.Checklist
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.