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

Completely specify initial conditions for DynamicPipeEnergyConservationCheck and DynamicPipeEnergyConservationCheck2 #4424

Open
casella opened this issue Jun 18, 2024 · 0 comments
Open

Completely specify initial conditions for DynamicPipeEnergyConservationCheck and DynamicPipeEnergyConservationCheck2 #4424

casella opened this issue Jun 18, 2024 · 0 comments
Assignees
@casella
Labels
L: ModelicaTest Issue addresses ModelicaTest, ModelicaTestConversion4 or ModelicaTestOverdetermined P: high High priority issue
Milestone
MSL4.1.0

Comments

@casella
Copy link
Contributor

casella commented Jun 18, 2024 •
edited
Loading

The models

ModelicaTest.Fluid.TestComponents.Pipes.DynamicPipeEnergyConservationCheck
ModelicaTest.Fluid.TestComponents.Pipes.DynamicPipeEnergyConservationCheck2

do not have fully specified initial conditions. These should be added to make sure that the initialization problem is not tool-dependent. This is probably also the reason why the first of the two models currently fails in OpenModelica.
@casella casella added L: ModelicaTest Issue addresses ModelicaTest, ModelicaTestConversion4 or ModelicaTestOverdetermined P: high High priority issue labels Jun 18, 2024
@casella casella added this to the MSL4.1.0 milestone Jun 18, 2024
@casella casella self-assigned this Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@casella casella

Labels
L: ModelicaTest Issue addresses ModelicaTest, ModelicaTestConversion4 or ModelicaTestOverdetermined P: high High priority issue
Projects
None yet
Milestone
MSL4.1.0
Development

No branches or pull requests
1 participant
@casella