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

Add support for multiple solutions #2100

Merged
merged 6 commits into from Nov 26, 2019
Merged

Add support for multiple solutions #2100

merged 6 commits into from Nov 26, 2019

Conversation

@odow
Copy link
Member

odow commented Nov 12, 2019

This is a WIP PR to add multiple objective support to JuMP.

Currently blocked by changed needed in MOI.

This PR adds support for querying multiple results form solvers via result_index.

src/constraints.jl Show resolved Hide resolved
odow added 2 commits Nov 22, 2019
@codecov

This comment has been minimized.

Copy link

codecov bot commented Nov 22, 2019

Codecov Report

Merging #2100 into master will increase coverage by 0.14%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2100      +/-   ##
==========================================
+ Coverage   90.95%   91.09%   +0.14%     
==========================================
  Files          38       40       +2     
  Lines        4321     4402      +81     
==========================================
+ Hits         3930     4010      +80     
- Misses        391      392       +1
Impacted Files Coverage Δ
src/variables.jl 95.05% <100%> (ø) ⬆️
src/JuMP.jl 82.11% <100%> (+1.13%) ⬆️
src/aff_expr.jl 88.54% <100%> (ø) ⬆️
src/optimizer_interface.jl 81.81% <100%> (+0.42%) ⬆️
src/nlp.jl 92.1% <100%> (+0.16%) ⬆️
src/objective.jl 91.17% <100%> (ø) ⬆️
src/constraints.jl 91.6% <100%> (ø) ⬆️
src/quad_expr.jl 94% <100%> (ø) ⬆️
src/indicator.jl 96% <0%> (ø)
src/callbacks.jl 100% <0%> (ø)
... and 2 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 b780023...0360717. Read the comment docs.

@odow odow changed the title DNMY: add support for multiple objectives Add support for multiple solutions Nov 22, 2019
src/constraints.jl Outdated Show resolved Hide resolved
src/constraints.jl Outdated Show resolved Hide resolved
src/constraints.jl Show resolved Hide resolved
docs/src/solutions.md Outdated Show resolved Hide resolved
src/constraints.jl Show resolved Hide resolved
odow added 2 commits Nov 24, 2019
@blegat
blegat approved these changes Nov 25, 2019
@mlubin
mlubin approved these changes Nov 26, 2019
src/constraints.jl Outdated Show resolved Hide resolved
@odow odow merged commit 326017c into master Nov 26, 2019
5 checks passed
5 checks passed
codecov/patch 100% of diff hit (target 90.95%)
Details
codecov/project 91.09% (+0.14%) compared to b780023
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@odow odow deleted the od/mo branch Nov 26, 2019
@blegat blegat mentioned this pull request Nov 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.