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

Error when selecting medium from the dropdown list #3970

Open
JayHuLBL opened this issue Aug 27, 2024 · 0 comments
Open

Error when selecting medium from the dropdown list #3970

JayHuLBL opened this issue Aug 27, 2024 · 0 comments
Assignees

Comments

@JayHuLBL
Copy link
Contributor

JayHuLBL commented Aug 27, 2024

It is to address the issue reported in IBPSA issue #1924.

In class Buildings.Fluid.HeatExchangers.ConstantEffectiveness, the choices given by the dropdown causes the code to try to redeclare Medium instead of Medium1 or Medium2, and see following error:

Screenshot 2024-08-27 at 12 14 20 PM

The root cause is from the below code section in baseclass Buildings.Fluid.Interfaces.PartialFourPort:

 replaceable package Medium1 =
    Modelica.Media.Interfaces.PartialMedium "Medium 1 in the component"
      annotation (choices(
        choice(redeclare package Medium = Buildings.Media.Air "Moist air"),
        choice(redeclare package Medium = Buildings.Media.Water "Water"),
        choice(redeclare package Medium =
            Buildings.Media.Antifreeze.PropyleneGlycolWater (
          property_T=293.15,
          X_a=0.40)
          "Propylene glycol water, 40% mass fraction")));
  replaceable package Medium2 =
    Modelica.Media.Interfaces.PartialMedium "Medium 2 in the component"
      annotation (choices(
        choice(redeclare package Medium = Buildings.Media.Air "Moist air"),
        choice(redeclare package Medium = Buildings.Media.Water "Water"),
        choice(redeclare package Medium =
            Buildings.Media.Antifreeze.PropyleneGlycolWater (
          property_T=293.15,
          X_a=0.40)
          "Propylene glycol water, 40% mass fraction")));

The change will merge to the branches, master, the 11, 10 and 9 maintenance branches. And the error needs to done for the class Buildings.Fluid.Interfaces.EightPort.

@JayHuLBL JayHuLBL self-assigned this Aug 27, 2024
@mwetter mwetter added this to the Release 12.0.0 milestone Sep 4, 2024
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

No branches or pull requests

2 participants