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

Nozzle.mo Documentation / v_out #42

Open
Flauipaui opened this issue Jan 18, 2022 · 4 comments
Open

Nozzle.mo Documentation / v_out #42

Flauipaui opened this issue Jan 18, 2022 · 4 comments
Assignees
Labels
layer::documentation Improvements or additions to model/class documentation

Comments

@Flauipaui
Copy link
Contributor

Flauipaui commented Jan 18, 2022

Hello,

while studying Processes/Nozzle.mo i have seen that the documentation at the end of the document is in my eyes not referring to the model, descriping flow resistances in pipes.

<p><span style=\"font-size: 12pt;\">Implementation of a flow resistance pipe with different selectable flow resistance functions (laminar, laminar-turbolent, linear-quadratic).</span></p>

Further, i have seen that you are modeling the flow veloctiy v_out with positive values in case of positive mass flows. I would have expected, as a convention, that the flow velocity v_out is negative since it is leaving the component. I have checked the code, the sign of the variables is not influencing the solution, though (used only: v_out^2). Changing the sign of v_out, thus rather fulfills a convention than correcting errors.

v_out = m_flow/(rho_out*A_out);

Let me know that you think.

Bests,

@mimeissner mimeissner added the layer::documentation Improvements or additions to model/class documentation label Jan 19, 2022
@mimeissner
Copy link
Collaborator

both points are true, thank you.
since they are not urgent, we will fix them at a later time.

@mimeissner mimeissner self-assigned this Jan 19, 2022
@Flauipaui
Copy link
Contributor Author

Hello,

I have just seen that in other models (see below), v_out is meant to be positive (equations based on v_out are designed for positive values of v_out). In case you are willing to introduce a convetion that v_out has negative values, you would need to adapt further models. I'm not sure, if that is in your interest ... and if there is a benefit for you.


It might be easier to add a coment, that v_out has positive values for outgoing flow.

Bests,

@tobolar tobolar added this to the 1.1.0 milestone Jun 27, 2023
@tobolar tobolar assigned nieweber and unassigned mimeissner Sep 5, 2023
@tobolar
Copy link
Contributor

tobolar commented Sep 5, 2023

The documentation of Nozzle was fixed with c34a0a4

@nieweber
Copy link
Contributor

Thank you again, your remarks on the sign convention are reasonable.
With PR #147 we have updated the documentation of the dynamic pressure components and added an explanation of the sign convention of the velocity. An internal review on changing the sign convention is still pending.

@tobolar tobolar modified the milestones: 1.1.0, next-MINOR-version Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
layer::documentation Improvements or additions to model/class documentation
Projects
None yet
Development

No branches or pull requests

4 participants