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

Steady State OGF Linear Relaxation #253

Merged
merged 7 commits into from
Dec 6, 2023
Merged

Steady State OGF Linear Relaxation #253

merged 7 commits into from
Dec 6, 2023

Conversation

kaarthiksundar
Copy link
Collaborator

The linear relaxations for non-linear terms in the OGF, migrated from @hskkanth's fork.

hskkanth and others added 6 commits December 6, 2023 11:22
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
Copy link

codecov bot commented Dec 6, 2023

Codecov Report

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

Comparison is base (7c8124a) 79.38% compared to head (32ef788) 79.43%.

Files Patch % Lines
src/prob/ogf_comp_power_proxy.jl 66.66% 20 Missing ⚠️
src/form/lrwp.jl 93.75% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #253      +/-   ##
==========================================
+ Coverage   79.38%   79.43%   +0.04%     
==========================================
  Files          37       38       +1     
  Lines        5365     5494     +129     
==========================================
+ Hits         4259     4364     +105     
- Misses       1106     1130      +24     

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

@kaarthiksundar kaarthiksundar merged commit 6a6c37a into master Dec 6, 2023
18 of 21 checks passed
@kaarthiksundar kaarthiksundar deleted the linear-ss branch December 6, 2023 22:02
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