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

[Bridges.Constraint] add bridges in alphabetical order #2596

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

odow
Copy link
Member

@odow odow commented Dec 21, 2024

No description provided.

@odow
Copy link
Member Author

odow commented Dec 30, 2024

I pulled out the .Variable stuff into #2599

I'll update this to focus only on .Constraint

@odow odow force-pushed the od/constraint-bridges branch 2 times, most recently from a74fddc to 950a540 Compare December 30, 2024 20:16
@odow odow changed the title [Bridges] explicitly document bridges not added by default [Bridges.Constraint] add bridges in alphabetical order Dec 30, 2024
@odow
Copy link
Member Author

odow commented Dec 30, 2024

@odow
Copy link
Member Author

odow commented Jan 1, 2025

@odow odow force-pushed the od/constraint-bridges branch from c498e19 to f0abce0 Compare January 2, 2025 22:18
# This bridge destorys a lot of structure and adding PSD variables is almost
# always undesirable. We give this bridge an arbitrarily hight cost so that it
# is used only if necessary.
bridging_cost(::Type{<:RSOCtoPSDBridge}) = 10.0
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@blegat, I've added these costs here too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants