Skip to content

Create a bash script to run mps files in parallel #87

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

Merged
merged 7 commits into from
Jul 20, 2025

Conversation

rg20
Copy link
Contributor

@rg20 rg20 commented Jun 10, 2025

This PR adds a bash script to run all mps files in a given directory in parallel. This should replace run_mip and run_pdlp scripts.

The script uses file locking to enable all free gpus to grab the next available mps file so that load balancing is ensured.

Copy link

copy-pr-bot bot commented Jun 10, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@rg20 rg20 added this to the 25.08 milestone Jun 12, 2025
@rg20 rg20 added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Jun 12, 2025
Copy link
Collaborator

@rgsl888prabhu rgsl888prabhu left a comment

Choose a reason for hiding this comment

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

@rg20 This is an awesome addition, just had few comments.

--mip-heuristics-only Run mip heuristics only
--write-log-file Write log file
--num-cpu-threads Number of CPU threads to use
--batch-num Batch number
Copy link
Collaborator

Choose a reason for hiding this comment

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

May be more details on batch number and n-batches and does it support LP or MIP or both.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need to expand more on these options ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added notes and example on the batch num

We have run_mip.cpp script that we have been using, I moved all the options from there here. If need more, we can add. But for now this is good, I think.

@rg20 rg20 force-pushed the benchmark_script branch from 6d7aaae to 3aae928 Compare July 18, 2025 13:27
@rg20
Copy link
Contributor Author

rg20 commented Jul 18, 2025

/ok to test 3aae928

@rg20
Copy link
Contributor Author

rg20 commented Jul 20, 2025

/merge

@rapids-bot rapids-bot bot merged commit b9f095e into NVIDIA:branch-25.08 Jul 20, 2025
345 of 352 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants