Skip to content

Conversation

@matbesancon
Copy link
Contributor

This PR upgrades Tulip to MOI 0.10
There are some minor issues to fix for the tests but not that many.

The blocker left for this PR is Convex.jl used in the tests and still locked on MOI 0.9

@matbesancon
Copy link
Contributor Author

Also closes #103, MOI 0.9 and 0.10 will not be compatible

@matbesancon
Copy link
Contributor Author

As a follow-up to this, the MOI tests could also be run with the numeric types supported b Tulip

@codecov-commenter
Copy link

codecov-commenter commented Oct 9, 2021

Codecov Report

Merging #105 (8eaa45a) into master (b0b3496) will decrease coverage by 1.97%.
The diff coverage is 83.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
- Coverage   90.64%   88.66%   -1.98%     
==========================================
  Files          43       43              
  Lines        2725     2718       -7     
==========================================
- Hits         2470     2410      -60     
- Misses        255      308      +53     
Impacted Files Coverage Δ
src/Interfaces/MOI/attributes.jl 79.71% <75.00%> (+8.06%) ⬆️
src/Interfaces/MOI/constraints.jl 92.80% <81.81%> (+2.07%) ⬆️
src/Interfaces/MOI/MOI_wrapper.jl 81.25% <100.00%> (+1.88%) ⬆️
src/Interfaces/MOI/objective.jl 100.00% <100.00%> (+7.69%) ⬆️
src/Interfaces/MOI/variables.jl 91.83% <100.00%> (+2.04%) ⬆️
src/Presolve/forcing_row.jl 16.10% <0.00%> (-55.94%) ⬇️
src/IPM/MPC/MPC.jl 87.50% <0.00%> (-1.98%) ⬇️
src/Presolve/row_singleton.jl 90.56% <0.00%> (-1.89%) ⬇️
src/IPM/HSD/step.jl 94.80% <0.00%> (-0.65%) ⬇️
... and 5 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 b0b3496...8eaa45a. Read the comment docs.

@matbesancon
Copy link
Contributor Author

bump @mtanneau

Copy link
Member

@mtanneau mtanneau left a comment

Choose a reason for hiding this comment

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

Good to go! Sorry for the delay

@mtanneau mtanneau merged commit 7c14ae9 into ds4dm:master Nov 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants