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

Add comparison signals for HBridge_TrianglePWM_RL, same as HBridge_RL #4423

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

casella
Copy link
Contributor

@casella casella commented Jun 18, 2024

As required by #4349.

@AHaumer the new model is very similar to HBridge_RL, so I just copied the comparisonSignals.txt of that model. Feel free to add more variables by committing to this PR, if you find this appropriate.

Once accepted, this needs to be back-ported to maint/4.1.0.

Copy link
Contributor

@maltelenz maltelenz left a comment

Choose a reason for hiding this comment

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

Looks good as far as I can tell.

@casella casella merged commit a9c6eab into modelica:master Jun 19, 2024
2 checks passed
@beutlich beutlich added L: Resources Issue addresses Modelica/Resources (excl. C-Sources) example Issue only addresses example(s) labels Jun 19, 2024
@beutlich beutlich changed the title Added comparison signals for HBridge_TrianglePWM_RL, same as HBridge_RL Add comparison signals for HBridge_TrianglePWM_RL, same as HBridge_RL Jun 19, 2024
@casella casella modified the milestone: MSL4.1.0 Jun 28, 2024
@Esther-Devakirubai
Copy link
Contributor

Backported to maintenance branch by #4454

@beutlich beutlich added this to the maintenance milestone Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
example Issue only addresses example(s) L: Resources Issue addresses Modelica/Resources (excl. C-Sources)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants