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

Problems with ModelicaTest.Fluid.TestComponents.Vessels.TestSimpleTank #4526

Open
casella opened this issue Feb 9, 2025 · 1 comment · May be fixed by #4527
Open

Problems with ModelicaTest.Fluid.TestComponents.Vessels.TestSimpleTank #4526

casella opened this issue Feb 9, 2025 · 1 comment · May be fixed by #4527
Assignees
Labels
bug Critical/severe issue L: Fluid Issue addresses Modelica.Fluid (excl. Dissipation)
Milestone

Comments

@casella
Copy link
Contributor

casella commented Feb 9, 2025

As reported here, ModelicaTest.Fluid.TestComponents.Vessels.TestSimpleTank is failing because the temperature of the fluid in the tank is getting out of range.

I investigated the issue in depth. The fundamental problem is that the base class Modelica.Fluid.Interfaces.PartialLumpedVolume is governed by a dynamic mass and energy balance, with a state variable change to more suitable state variables than mass and internal energy, in this case level and fluid temperature.

It is clear that the energy balance equation becomes singular when the level (hence the mass) becomes zero. Don't even mention what happens when the mass becomes negative (anti-matter anyone?). Some solver could miraculously handle the singular case without producing utter garbage, but I don't think that the MSL should rely on miracles, it should only contain solid, well-posed models that can safely be handle by all MLS-compliant tools.

Bottom line: the tank should never get completely empty.

@casella casella added bug Critical/severe issue L: Fluid Issue addresses Modelica.Fluid (excl. Dissipation) labels Feb 9, 2025
@casella casella added this to the MSL4.1.0 milestone Feb 9, 2025
@casella casella self-assigned this Feb 9, 2025
@casella
Copy link
Contributor Author

casella commented Feb 9, 2025

See PR #4527. If successful, will need to be back-ported to maint/v4.1.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Critical/severe issue L: Fluid Issue addresses Modelica.Fluid (excl. Dissipation)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant