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

MPS #497

Merged
merged 23 commits into from
Apr 3, 2025
Merged

MPS #497

merged 23 commits into from
Apr 3, 2025

Conversation

DLWoodruff
Copy link
Collaborator

Support for using MPS files to get scenarios (with json files that provide tree information)

DLWoodruff and others added 18 commits March 13, 2025 13:40
@DLWoodruff
created an mps readers based on pulp
3b3dc6d
@DLWoodruff
created an mps_module to allow for mps based scenarios
37d04c5
@DLWoodruff
[WIP] added to the format for json files to accompany lp and mps files
a222aea
@DLWoodruff
can now read mps using coin mip; here's a little demo
6fd3bcf
@DLWoodruff
major corrections by dlw
04472ad
@DLWoodruff
mps_coin_mip.py executes, but the solutions don't match
af84d03
@DLWoodruff
the mps reader is ready for testing
de8a5cc
@DLWoodruff
added a test for the mps_reader
038675e
@DLWoodruff
[WIP] getting start on testing for the mps module option in generic c…
43d5d0c 
…ylinders
@DLWoodruff
[WIP] working on testin generic tester and fixing bugs (still need to…
4c039b1 
… work on cfg.num_scens
@DLWoodruff
working on generic tester to get cfg.num_scens taken care of
cd6d759
@DLWoodruff
there is now a test for mps reader and mps module in generic tester, …
0095e05 
…but it needs to be easier for xpress
@DLWoodruff
touch up tests
ed4be65
@DLWoodruff
added a demonstration and some documentation.
79072bd
@DLWoodruff
tests are ready to run?
a8aba60
@DLWoodruff
cleanup
ea8ad8c
@DLWoodruff
trying to debug the admm_wrapper test
93bdd0c
@DLWoodruff
Update test_admmWrapper.py
d16a7d4
@DLWoodruff DLWoodruff requested a review from bknueven March 20, 2025 01:27
@DLWoodruff DLWoodruff marked this pull request as draft March 20, 2025 23:16
@DLWoodruff DLWoodruff marked this pull request as ready for review March 21, 2025 00:54
bknueven
bknueven approved these changes Mar 26, 2025
View reviewed changes
Copy link
Collaborator

@bknueven bknueven left a comment

Choose a reason for hiding this comment

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

I few minor things; nothing to hold this up.

@DLWoodruff DLWoodruff enabled auto-merge (squash) April 3, 2025 18:35
@DLWoodruff DLWoodruff merged commit e697a1b into Pyomo:main Apr 3, 2025
19 checks passed
@DLWoodruff DLWoodruff deleted the mps branch April 3, 2025 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bknueven bknueven bknueven approved these changes

@jeanpaulwatson jeanpaulwatson Awaiting requested review from jeanpaulwatson

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DLWoodruff @bknueven