-
Notifications
You must be signed in to change notification settings - Fork 546
Update the PyROS Separation Priority Ordering Interface #3581
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
base: main
Are you sure you want to change the base?
Update the PyROS Separation Priority Ordering Interface #3581
Conversation
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.
This looks good!
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.
Looks good!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3581 +/- ##
=======================================
Coverage 88.92% 88.92%
=======================================
Files 888 888
Lines 102347 102391 +44
=======================================
+ Hits 91009 91054 +45
+ Misses 11338 11337 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary/Motivation:
Update the PyROS solver interface for specification of constraint separation priorities.
Changes proposed in this PR:
Suffix
components with local name"pyros_separation_priority"
separation_priority_order
Var
orConstraint
component declared on the model. Thus, for example, constraints derived fromVar
bound specifications can be prioritized by mapping priorities toVar
componentsNone
to be specified as a priority for equality and inequality constraints that are not to be robustly enforced. These constraints are enforced subject to only the nominal uncertain parameter realization, and are therefore excluded from the separation problems. Note also that equality constraints with a priority ofNone
are, therefore, not reformulated via coefficient matching or discretization, where otherwise applicableTODO
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: