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

Conditional connection is not displayed #11202

Closed
FWuellhorst opened this issue Sep 18, 2023 · 5 comments · Fixed by #11210
Closed

Conditional connection is not displayed #11202

FWuellhorst opened this issue Sep 18, 2023 · 5 comments · Fixed by #11210
Assignees
Labels
COMP/GUI/InstBased Interface Issues with new instantiation-based OMEdit feature COMP/GUI/OMEdit Issue and pull request related to OMEdit
Milestone

Comments

@FWuellhorst
Copy link

FWuellhorst commented Sep 18, 2023

Description

Conditional connections are not showing.

Steps to Reproduce

Create this model

model ConnectionNotShowing
  parameter Boolean useConst=true;
  Modelica.Blocks.Sources.Constant const annotation(
    Placement(visible = true, transformation(origin = {-10, 10}, extent = {{-10, -10}, {10, 10}}, rotation = 0)));
  Modelica.Blocks.Interfaces.RealOutput y annotation(
    Placement(visible = true, transformation(origin = {110, 0}, extent = {{-10, -10}, {10, 10}}, rotation = 0), iconTransformation(origin = {72, 14}, extent = {{-10, -10}, {10, 10}}, rotation = 0)));
equation
  if useConst then
    connect(const.y, y) annotation(
    Line(points = {{2, 10}, {78, 10}, {78, 0}, {110, 0}}, color = {0, 0, 127}));
  end if;
annotation(
    uses(Modelica(version = "4.0.0")));
end ConnectionNotShowing;

and look at the missing connection.

Expected Behavior

The connection is shown.

Screenshots

image

Version and OS

Connected to OpenModelica v1.22.0-dev-197-g9166c1a4da (64-bit)
Connected to OMSimulator v2.1.1.post188-gaf996ad-mingw

  • OS: Windows 10, 64 bit
  • Modelica 4.0.0

Additional Context

We are going to use OM for the IBPSA workshop at the Modelica Conference.
This issue could confuse users looking at the solution in the IBPSA.
If a quick fix is possible, it would be nice for the workshop.
Refer to the workshop PR: ibpsa/modelica-ibpsa#1788

@adeas31
Copy link
Member

adeas31 commented Sep 18, 2023

@perost I guess its easy to fix.

@adeas31 adeas31 added this to the 1.22.0 milestone Sep 18, 2023
@casella
Copy link
Contributor

casella commented Sep 18, 2023

@perost will check later today.

@casella casella added COMP/GUI/InstBased Interface Issues with new instantiation-based OMEdit feature COMP/GUI/OMEdit Issue and pull request related to OMEdit labels Sep 18, 2023
perost added a commit to perost/OpenModelica that referenced this issue Sep 18, 2023
perost added a commit to perost/OpenModelica that referenced this issue Sep 18, 2023
@FWuellhorst
Copy link
Author

Thanks for the quick fix! :)

@casella
Copy link
Contributor

casella commented Sep 18, 2023

@FWuellhorst, you can get the fix in the latest nightly build already.

@FWuellhorst
Copy link
Author

@casella @perost It works, thanks for the fix :) See you in Aachen!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
COMP/GUI/InstBased Interface Issues with new instantiation-based OMEdit feature COMP/GUI/OMEdit Issue and pull request related to OMEdit
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants