Skip to content

Feat: Update handling of diverging or converging pipe.#69

Merged
dietmarw merged 34 commits into
masterfrom
bb_small_change_01
Feb 2, 2026
Merged

Feat: Update handling of diverging or converging pipe.#69
dietmarw merged 34 commits into
masterfrom
bb_small_change_01

Conversation

@boerrebj
Copy link
Copy Markdown
Collaborator

@boerrebj boerrebj commented Jan 29, 2026

Fixes: #63

Update the handling of pipes with different inlet and outlet diameter.

  • Changed the initialization of pipe flow.

@github-actions github-actions Bot added the feat label Jan 29, 2026
@dietmarw dietmarw added this to the v3.0.0 milestone Jan 29, 2026
dietmarw and others added 27 commits February 2, 2026 11:56
 ref: #63

   This  extends the simple pipe momentum equation to include
      a local loss term F_taper = K_c · (ρ·v_o^2/2)·A_o for the diameter contraction.
      Adjust K_c based on your taper geometry: 0.05–0.15 for gentle cones,
      up to 0.6 for sharp contractions.
WORK IN PROGRESS - NOT YET COMPLETE
 ref: #63

   This  extends the simple pipe momentum equation to include
      a local loss term F_taper = K_c · (ρ·v_o^2/2)·A_o for the diameter contraction.
      Adjust K_c based on your taper geometry: 0.05–0.15 for gentle cones,
      up to 0.6 for sharp contractions.
WORK IN PROGRESS - NOT YET COMPLETE
Copy link
Copy Markdown
Collaborator

@dietmarw dietmarw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Se inline comments

Comment thread OpenHPL/Examples/Pipes.mo Outdated
Waterway.Reservoir Upstream(h_0 = 89.67, constantLevel=true)
annotation(
Placement(transformation(origin={-50,0}, extent={{-10,-10},{10,10}})));
Waterway.Reservoir Downstream(h_0 = -10.33, constantLevel=true)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we now can set the pressure reference to gauge pressure. Would it not be a good idea to use that also in this example?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. Agree.

Comment thread OpenHPL/Waterway/Pipe.mo Outdated
Comment thread README.md Outdated
@dietmarw dietmarw dismissed their stale review February 2, 2026 13:32

all fixed now.

@dietmarw dietmarw merged commit 9c163ba into master Feb 2, 2026
6 checks passed
@dietmarw dietmarw self-assigned this Feb 2, 2026
@dietmarw dietmarw deleted the bb_small_change_01 branch February 2, 2026 13:34
@dietmarw
Copy link
Copy Markdown
Collaborator

dietmarw commented Feb 2, 2026

Thanks, it's nice to see this fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pipe model calculates unreasonable pressure drop when different diameters are used

2 participants