-
Notifications
You must be signed in to change notification settings - Fork 517
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
[contrib] Add plot
method to IncidenceGraphInterface
#2716
Conversation
plot
method to IncidenceGraphInterface
plot
method to IncidenceGraphInterface
Marking as WIP until #2715 is merged. |
plot
method to IncidenceGraphInterface
plot
method to IncidenceGraphInterface
def plot(self, variables=None, constraints=None, title=None, show=True): | ||
"""Plot the bipartite incidence graph of variables and constraints | ||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoop!
Codecov ReportBase: 87.05% // Head: 86.99% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #2716 +/- ##
==========================================
- Coverage 87.05% 86.99% -0.06%
==========================================
Files 771 771
Lines 86257 86301 +44
==========================================
- Hits 75091 75080 -11
- Misses 11166 11221 +55
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
EDIT: Dependence of #2715, #2716, #2723, and #2727
plot
method toIncidenceGraphInterface
#2716 incorporates [contrib] Refactorincidence_analysis
to cache a graph instead of a matrix #2715plot
method toIncidenceGraphInterface
#2716 and Adding basic diagnostics output to solve_strongly_connected_components #2723In summary, merging #2727 will incorporate all the others, but they are broken up to be easier to review.
Summary/Motivation:
@michaelbynum wrote a nice method for plotting the incidence graph of a Pyomo model in #2251, so I shamelessly stole it and added it as a method on
IncidenceGraphInterface
.Changes proposed in this PR:
plot
method toIncidenceGraphInterface
Note that this PR depends on #2715.
Legal Acknowledgement
By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution: