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/fix description comments in Fluid #3281

Merged
merged 1 commit into from
Jan 7, 2020

Conversation

beutlich
Copy link
Member

Fix the remaining parameter descriptions to finally resolve #2636.

@beutlich beutlich added the task General work that is not related to a bug or feature label Dec 21, 2019
@beutlich beutlich added this to the MSL4.0.0 milestone Dec 21, 2019
@beutlich beutlich self-assigned this Dec 21, 2019
@beutlich beutlich added the L: Fluid Issue addresses Modelica.Fluid (excl. Dissipation) label Dec 21, 2019
@beutlich beutlich requested a review from thorade January 4, 2020 19:12
@beutlich
Copy link
Member Author

beutlich commented Jan 5, 2020

@casella OK for you?

annotation (Placement(transformation(
extent={{-20,0},{20,-10}},
origin={0,-100})));

// Heat transfer through boundary
parameter Boolean use_HeatTransfer = false
"= true to use the HeatTransfer model"
"= true, if HeatTransfer model is used"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think I liked the previous version of this comment better

@beutlich beutlich removed the request for review from casella January 7, 2020 16:53
@beutlich beutlich merged commit 565849f into modelica:master Jan 7, 2020
@beutlich beutlich deleted the fix-param-comments branch January 7, 2020 16:57
@casella
Copy link
Contributor

casella commented Jan 9, 2020

For the record, I think this is really no big deal 😃

@casella
Copy link
Contributor

casella commented Jan 9, 2020

If it's fine for @thorade, it's fine for me 😃

Sorry, but I took two weeks off for the Christmas break

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: Fluid Issue addresses Modelica.Fluid (excl. Dissipation) task General work that is not related to a bug or feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Parameters' description string missing or not starting with upper case
4 participants