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.

Sign up for GitHub

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

Jump to bottom

SimplexIterations uses deprecated method #86

Closed
odow opened this issue Feb 21, 2022 · 1 comment · Fixed by #87
Closed

SimplexIterations uses deprecated method #86

odow opened this issue Feb 21, 2022 · 1 comment · Fixed by #87
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed Wrapper: MathOptInterface

Comments

@odow
Copy link
Member

odow commented Feb 21, 2022

HiGHS.jl/src/MOI_wrapper.jl

Lines 1792 to 1794 in 94ae3b5

function MOI.get(model::Optimizer, ::MOI.SimplexIterations)
return Highs_getSimplexIterationCount(model)
end

should be replaced with lookups for

https://github.com/ERGO-Code/HiGHS/blob/17237a320d5c32a95c18e1e8cf2ef4a365e9e889/src/lp_data/HighsInfo.h#L200-L205
@odow odow added bug Something isn't working help wanted Extra attention is needed good first issue Good for newcomers Wrapper: MathOptInterface labels Feb 21, 2022
@odow odow mentioned this issue Feb 21, 2022
Merged
@odow
Copy link
Member Author

odow commented Feb 21, 2022 •
edited
Loading

We should also add mip_node_count.

Blocked by ERGO-Code/HiGHS#747

This was referenced Feb 21, 2022
Closed
Closed
@odow odow closed this as completed in #87 Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed Wrapper: MathOptInterface
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix SimplexIterations getter and add BarrierIterations jump-dev/HiGHS.jl
1 participant
@odow