Skip to content

Fix bug on consecutive runs caused by static lp_state_t object#24

Merged
rapids-bot[bot] merged 1 commit intoNVIDIA:branch-25.05from
aliceb-nv:lp-state-bugfix
May 21, 2025
Merged

Fix bug on consecutive runs caused by static lp_state_t object#24
rapids-bot[bot] merged 1 commit intoNVIDIA:branch-25.05from
aliceb-nv:lp-state-bugfix

Conversation

@aliceb-nv
Copy link
Copy Markdown
Contributor

@aliceb-nv aliceb-nv commented May 21, 2025

This PR fixes occasional crashes seen in the regression tests caused by accesses to a global lp_state_t object across consecutive runs, becoming stale in later runs.

The object has been moved to mip_solver_context_t.

@aliceb-nv aliceb-nv added this to the 25.05 milestone May 21, 2025
@aliceb-nv aliceb-nv requested a review from a team as a code owner May 21, 2025 15:55
@aliceb-nv aliceb-nv requested review from akifcorduk and kaatish May 21, 2025 15:55
@aliceb-nv aliceb-nv added bug Something isn't working non-breaking Introduces a non-breaking change labels May 21, 2025
@aliceb-nv
Copy link
Copy Markdown
Contributor Author

/ok to test

@aliceb-nv aliceb-nv removed the request for review from kaatish May 21, 2025 18:16
@rg20
Copy link
Copy Markdown
Contributor

rg20 commented May 21, 2025

/merge

@rapids-bot rapids-bot bot merged commit c796935 into NVIDIA:branch-25.05 May 21, 2025
85 checks passed
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
…A#24)

This PR fixes occasional crashes seen in the regression tests caused by accesses to a global `lp_state_t` object across consecutive runs, becoming stale in later runs.

The object has been moved to `mip_solver_context_t`.

Authors:
  - Alice Boucher (https://github.com/aliceb-nv)

Approvers:
  - Rajesh Gandham (https://github.com/rg20)

URL: NVIDIA#24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants