-
Notifications
You must be signed in to change notification settings - Fork 561
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
Update the PyROS Separation Priority Ordering Interface #3581
Conversation
jsiirola
left a comment
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!
emma58
left a comment
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 Report✅ All 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:
Suffixcomponents with local name"pyros_separation_priority"separation_priority_orderVarorConstraintcomponent declared on the model. Thus, for example, constraints derived fromVarbound specifications can be prioritized by mapping priorities toVarcomponentsNoneto 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 ofNoneare, 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: