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

Linear approximation for pipe headloss and new problem for simulations #254

Merged
merged 7 commits into from
Dec 14, 2023

Conversation

hskkanth
Copy link
Collaborator

Added a linear approximation for pipe headloss.
Replaced the problem "ogf_comp_power_proxy" with ogf_comp_power_pipe_proxy" for use with the simulator.
All tests passed

Pressure is supposed to decrease with withdrawal of gas
Added lrwp version of the pipe weymouth constraint.

Created a new ogf problem by replacing the compressor power objective with the difference in the squared pressures at the compressor end points
Replacing the nonlinear compressor power objective with a linear proxy, which is the absolute value of the pressure difference at the compressors endpoints
Updating the description of the new objective that uses a proxy for the compressor power
Adding a linear approximation for pipe head loss.
Replacing the problem  ogf_compr_power_proxy by ogf_comp_power_and_pipe_proxy to help the simulator
Copy link

codecov bot commented Dec 13, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (6a6c37a) 79.43% compared to head (133035e) 79.25%.

Files Patch % Lines
src/core/constraint.jl 80.00% 2 Missing ⚠️
src/prob/ogf_comp_power_and_pipe_proxy.jl 60.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #254      +/-   ##
==========================================
- Coverage   79.43%   79.25%   -0.18%     
==========================================
  Files          38       38              
  Lines        5494     5519      +25     
==========================================
+ Hits         4364     4374      +10     
- Misses       1130     1145      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kaarthiksundar kaarthiksundar merged commit 133035e into lanl-ansi:master Dec 14, 2023
9 of 12 checks passed
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

Successfully merging this pull request may close these issues.

2 participants