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.

Sign up for GitHub

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

Jump to bottom

Relocate UsersGuide.ParameterDefaults #3248

Closed
tobolar opened this issue Dec 3, 2019 · 1 comment · Fixed by #3256
Closed

Relocate UsersGuide.ParameterDefaults #3248

tobolar opened this issue Dec 3, 2019 · 1 comment · Fixed by #3256
Assignees
@christiankral @tobolar @beutlich
Labels
documentation Issue addresses the documentation L: UsersGuide Issue addresses Modelica.UsersGuide task General work that is not related to a bug or feature
Milestone
MSL4.0.0

Comments

@tobolar
Copy link
Contributor

tobolar commented Dec 3, 2019 •
edited

Modelica.UsersGuide.ParameterDefaults (Parameter defaults) summarises the convention how to handle default parameters. IMO this shall be a part of Modelica.UsersGuide.Conventions or even of Modelica.UsersGuide.Conventions.ModelicaCode instead. I.e. the class shall be moved to either of the packages.
@tobolar tobolar added L: UsersGuide Issue addresses Modelica.UsersGuide discussion Discussion issue that it not necessarily related to a concrete bug or feature documentation Issue addresses the documentation labels Dec 3, 2019
@beutlich
Copy link
Member

beutlich commented Dec 4, 2019

Either is fine for me.

@christiankral christiankral mentioned this issue Dec 7, 2019
Merged
@beutlich beutlich added this to the MSL4.0.0 milestone Dec 10, 2019
beutlich added a commit to christiankral/ModelicaStandardLibrary that referenced this issue Dec 21, 2019
@beutlich
refs modelica#3248: Add conversion
b210623
beutlich added a commit to christiankral/ModelicaStandardLibrary that referenced this issue Dec 21, 2019
@beutlich
refs modelica#3248: Add conversion
2b6a4ab
beutlich added a commit to christiankral/ModelicaStandardLibrary that referenced this issue Dec 21, 2019
@beutlich
refs modelica#3248: Fix merge conflict
c07b813
@beutlich beutlich added task General work that is not related to a bug or feature and removed discussion Discussion issue that it not necessarily related to a concrete bug or feature labels Dec 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@christiankral christiankral

@tobolar tobolar

@beutlich beutlich

Labels
documentation Issue addresses the documentation L: UsersGuide Issue addresses Modelica.UsersGuide task General work that is not related to a bug or feature
Projects
None yet
Milestone
MSL4.0.0
Development

Successfully merging a pull request may close this issue.

Relocate ParameterDefaults christiankral/ModelicaStandardLibrary
3 participants
@christiankral @tobolar @beutlich