Skip to content
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

Make propagation final of lifted parameters in Spring #4520

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

maltelenz
Copy link
Contributor

Backport #4407 to 4.1.0.

* Make propagation final of lifted parameter 'c' in Spring

Fixes modelica#3707.

* Make propagation final of lifted parameter 's_rel0' in Spring

* Make propagation final of lifted parameters in Spring's lineForce component

Even though the modification of lineShapeType is not parameter lifting, it would make little sense to make it non-final when all other modifications are final.

* Make propagation of lifted parameter 'd' final in ModelicaTest example SpringDamperParallel

---------

Co-authored-by: Anton Haumer <[email protected]>
(cherry picked from commit a5cf0f3)
@maltelenz maltelenz added this to the MSL4.1.0 milestone Jan 21, 2025
@maltelenz maltelenz requested review from casella and AHaumer January 21, 2025 11:13
@maltelenz maltelenz added the L: Mechanics.MultiBody Issue addresses Modelica.Mechanics.MultiBody label Jan 21, 2025
@maltelenz
Copy link
Contributor Author

@casella I'm not authorized to merge to the maint branch, so please press the button (or anyone else who is allowed).

@HansOlsson
Copy link
Contributor

I checked that they seem ok in MSL.

Note that there is a hierarchical modifier for Spring's lineForce in Modelica.Mechanics.MultiBody.Examples.Elementary.ThreeSprings but it is for r_rel_0, not any of the ones with final modifiers (it could also be moved to r_rel_0 directly in Spring). That is one reason we shouldn't apply final-modifiers everywhere yet.

Copy link
Contributor

@AHaumer AHaumer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@AHaumer AHaumer merged commit d2592ea into modelica:maint/4.1.x Jan 21, 2025
6 checks passed
@beutlich beutlich changed the title Make propagation final of lifted parameters in Spring (backport) Make propagation final of lifted parameters in Spring Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: Mechanics.MultiBody Issue addresses Modelica.Mechanics.MultiBody
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants