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 initial setup for GenX #4

Merged
merged 2 commits into from
Sep 11, 2024
Merged

Add initial setup for GenX #4

merged 2 commits into from
Sep 11, 2024

Conversation

odow
Copy link
Member

@odow odow commented Sep 10, 2024

So I think collecting the benchmark instances would be made much easier by
jump-dev/HiGHS.jl#224

function HiGHS.Highs_run(highs)
_write_highs_model(highs)
return ccall((:Highs_run, HiGHS.libhighs), Cint, (Ptr{Cvoid},), highs)
end
Copy link
Member Author

Choose a reason for hiding this comment

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

⛵ 🏴‍☠️ ☠️

@joaquimg thoughts on this?

@odow
Copy link
Member Author

odow commented Sep 11, 2024

So GenX has
https://github.com/GenXProject/GenX.jl/blob/150fe7af80410a11bd17adf544c186f898f0ddcd/src/model/generate_model.jl#L262-L266

but I think it's best if we just intercept HiGHS.

I'm running a full sweep now.

@odow odow mentioned this pull request Sep 11, 2024
@odow odow changed the title WIP: Add initial setup for GenX Add initial setup for GenX Sep 11, 2024
@odow
Copy link
Member Author

odow commented Sep 11, 2024

I've added some instances for now. But I'll make sure to rebase merge them so that the commits stay separate.

@odow odow merged commit 1537ce1 into main Sep 11, 2024
@odow odow deleted the od/genx branch September 11, 2024 07:56
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.

1 participant