refactor: kwargs of base system are now kwargs of main component

[ven-k]

• With this Symbolic defaults for sub-components and base system kwargs + set base sys kwargs as component's kwargs  ModelingToolkit.jl#2226, kwargs of base system as kwargs of main sys.
• Symbolic defaults can be set. Hence use_support is by default false everywhere.
• Tests are updated accordingly.
• Depends on Symbolic defaults for sub-components and base system kwargs + set base sys kwargs as component's kwargs  ModelingToolkit.jl#2226

[YingboMa]

We should add a bound on MTK as well.

[codecov]

Codecov Report

Merging #209 (3f63e66) into main (a99b91d) will increase coverage by 0.10%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #209      +/-   ##
==========================================
+ Coverage   55.00%   55.10%   +0.10%     
==========================================
  Files          48       48              
  Lines        1620     1617       -3     
==========================================
  Hits          891      891              
+ Misses        729      726       -3     

Files Changed	Coverage Δ
src/Blocks/nonlinear.jl	84.61% <ø> (ø)
src/Electrical/Analog/ideal_components.jl	0.00% <0.00%> (ø)
src/Mechanical/Rotational/components.jl	54.54% <ø> (ø)
src/Mechanical/Rotational/sources.jl	71.42% <ø> (ø)
src/Mechanical/Translational/components.jl	47.72% <ø> (ø)
src/Mechanical/TranslationalModelica/components.jl	0.00% <ø> (ø)
src/Mechanical/TranslationalPosition/utils.jl	22.22% <ø> (ø)
src/Blocks/math.jl	100.00% <100.00%> (+5.00%)	⬆️
src/Blocks/sources.jl	65.38% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[ven-k]

[See revised conclusions below]
Error with invalidations test seem to be sporadic.
MSL fails to precompile in the Invalidations test; while it precompiles and passes unit-tests.
I couldn't recreate the error locally for this snippet¹.
Re-running it might just pass it.

Note that the error

ERROR: LoadError: syntax: function argument name not unique: "y_start" around /home/runner/work/ModelingToolkitStandardLibrary.jl/ModelingToolkitStandardLibrary.jl/src/Blocks/nonlinear.jl:77

shouldn't appear for this branch. y_start has been removed from @parametersblock ofSlewRateLimiter` here².
It appears to be some caching issue; however, even after digging through the logs I couldn't find any evidence for that

Footnotes

1. https://github.com/SciML/ModelingToolkitStandardLibrary.jl/actions/runs/6075562148/job/16481994939#step:8:64 ↩

2. https://github.com/SciML/ModelingToolkitStandardLibrary.jl/pull/209/files#diff-4c7a873472d2ebf3629882dc4228cc2de7a094396b71f42ef05f1a4bbd1805e7R107~ ↩

[ven-k]

In the invalidations test, this branch successfully precompiles- see¹.
However it is the default branch (against which invalidations count is taken), fails to precompile. That should be expected with [email protected] and this PR is meant to fix the errors like:

ERROR: LoadError: syntax: function argument name not unique: "y_start" around /home/runner/work/ModelingToolkitStandardLibrary.jl/ModelingToolkitStandardLibrary.jl/src/Blocks/nonlinear.jl:77

So this PR can be merged.

Footnotes

1. https://github.com/SciML/ModelingToolkitStandardLibrary.jl/actions/runs/6079874565/job/16492986525#step:5:292 ↩