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

Appendix D uses notation '\dot{m}' instead of 'm_flow' #2644

Closed
henrikt-ma opened this issue Sep 3, 2020 · 1 comment · Fixed by #2645
Closed

Appendix D uses notation '\dot{m}' instead of 'm_flow' #2644

henrikt-ma opened this issue Sep 3, 2020 · 1 comment · Fixed by #2645

Comments

@henrikt-ma
Copy link
Collaborator

Quoting @HansOlsson in #2643:

… but I realized that there is a major issue in the appendix D: it is using \dot{m} instead of m_flow throughout - without the considerations I gave, whereas chapter 15 uses m_flow. Apart from the problem I stated it's also confusing to have different notations with no explanation.

On the other hand \dot{m} is more common mathematical notation, it's just misleading. I believe we can resolve it by introducing \tilde{m} (or something like that) for m_flow - and have a short explanation for the relation between \tilde{m}and \dot{m}.

henrikt-ma added a commit to henrikt-ma/ModelicaSpecification that referenced this issue Sep 3, 2020
Fixes modelica#2643.

Note, however, that the figure will probably need another update when fixing modelica#2644.
HansOlsson added a commit to HansOlsson/ModelicaSpecification that referenced this issue Sep 4, 2020
The reason is that it's the sum of mass flow rates that give \dot{m}
and the terms are not mass-derivatives.
Closes modelica#2644
@henrikt-ma
Copy link
Collaborator Author

This comment from #2643 could also be of interest here, quoting @HansOlsson:

I finally found the explanation.

Rüdiger Franke did a presentation "Excerpt of Motivation for Modelica stream connectors" as part of "Eurosyslib WP 5.3 Meeting Mannheim, June 9, 2008" (stored in Modelica33 on old svn-server), and used Z instead of m_flow, and included that figure as well.

HansOlsson added a commit that referenced this issue Oct 5, 2020
* Change notation for mass flow rate from \dot{m} to \tilde{m}.
The reason is that it's the sum of mass flow rates that give \dot{m}
and the terms are not mass-derivatives.
Closes #2644

Co-authored-by: Henrik Tidefelt <henrikt@wolfram.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant