Skip to content

Conversation

blegat
Copy link
Member

@blegat blegat commented Jun 3, 2019

This tidy up is needed before adding objective (#529) and variable bridges (#710).
This is breaking for bridges as MOI.Bridges.AbstractBridge is now MOI.Bridges.Constraint.AbstractBridge. Same thing for concrete_bridge_type and added_constraint_types.

@blegat blegat added the breaking label Jun 3, 2019
@blegat blegat added this to the v0.9 milestone Jun 3, 2019
@codecov-io
Copy link

Codecov Report

Merging #756 into master will increase coverage by <.01%.
The diff coverage is 86.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #756      +/-   ##
==========================================
+ Coverage   94.32%   94.33%   +<.01%     
==========================================
  Files          56       57       +1     
  Lines        6225     6226       +1     
==========================================
+ Hits         5872     5873       +1     
  Misses        353      353
Impacted Files Coverage Δ
src/Bridges/Constraint/square.jl 98.27% <ø> (ø)
src/Bridges/Constraint/functionize.jl 95% <ø> (ø)
src/Bridges/Constraint/soc_to_psd.jl 100% <ø> (ø)
src/Bridges/Constraint/scalarize.jl 59.45% <ø> (ø)
src/Bridges/Constraint/rsoc.jl 100% <ø> (ø)
src/Bridges/Constraint/geomean.jl 100% <ø> (ø)
src/Bridges/Constraint/indicator.jl 100% <ø> (ø)
src/Bridges/Constraint/flip_sign.jl 95.65% <ø> (ø)
src/Bridges/Constraint/interval.jl 86.84% <ø> (ø)
src/Bridges/Constraint/vectorize.jl 82.5% <ø> (ø)
... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 50d9118...79c887a. Read the comment docs.

@blegat blegat merged commit 7cec733 into master Jun 8, 2019
@odow odow deleted the bl/constraint_bridge branch June 15, 2019 15:44
coroa added a commit to coroa/JuMP.jl that referenced this pull request Jul 8, 2019
coroa added a commit to coroa/JuMP.jl that referenced this pull request Aug 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants