Skip to content

Decompression for .mps.gz and .mps.bz2 files - #357

Merged
rapids-bot[bot] merged 13 commits into
NVIDIA:branch-25.10from
ahehn-nv:ahehn/libmps_parser_compression
Sep 3, 2025
Merged

rapids-bot[bot] merged 13 commits into
NVIDIA:branch-25.10from
ahehn-nv:ahehn/libmps_parser_compression

Conversation

@ahehn-nv

Copy link
Copy Markdown
Contributor

Description

Adding zlib and bzip2 decompression to mps_parser, such that .mps.gz and .mps.bz2 files can be opened directly.

Checklist

  • I am familiar with the Contributing Guidelines.
  • Testing
    • New or existing tests cover these changes
    • Added tests
    • Created an issue to follow-up
    • NA
  • Documentation
    • The documentation is up to date with these changes
    • Added new documentation
    • NA

@copy-pr-bot

copy-pr-bot Bot commented Aug 28, 2025

Copy link
Copy Markdown

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.

Comment thread cpp/libmps_parser/CMakeLists.txt
Co-authored-by: Kyle Edwards <kyedwards@nvidia.com>
@anandhkb anandhkb added this to the 25.10 milestone Aug 28, 2025
@ramakrishnap-nv ramakrishnap-nv added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Aug 28, 2025
@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/ok to test 0672b41

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/ok to test 4cc3a6c

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/ok to test 6d54acf

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/ok to test d050010

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/ok to test 50abd29

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/ok to test 823b28f

@Kh4ster Kh4ster left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Super useful addition and great code quality, many thanks!

@aliceb-nv aliceb-nv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, approving :)

@aliceb-nv

Copy link
Copy Markdown
Contributor

/ok to test 01f133b

@Iroy30 Iroy30 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/merge

@rapids-bot
rapids-bot Bot merged commit 536488d into NVIDIA:branch-25.10 Sep 3, 2025
73 checks passed
@ahehn-nv
ahehn-nv deleted the ahehn/libmps_parser_compression branch September 8, 2025 11:45
aliceb-nv pushed a commit that referenced this pull request Sep 22, 2025
Adding zlib and bzip2 decompression to mps_parser, such that .mps.gz and .mps.bz2 files can be opened directly.

Authors:
  - https://github.com/ahehn-nv
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)
  - Nicolas Blin (https://github.com/Kh4ster)
  - Alice Boucher (https://github.com/aliceb-nv)
  - Ishika Roy (https://github.com/Iroy30)

URL: #357
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
Adding zlib and bzip2 decompression to mps_parser, such that .mps.gz and .mps.bz2 files can be opened directly.

Authors:
  - https://github.com/ahehn-nv
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)
  - Nicolas Blin (https://github.com/Kh4ster)
  - Alice Boucher (https://github.com/aliceb-nv)
  - Ishika Roy (https://github.com/Iroy30)

URL: NVIDIA#357
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.

7 participants