Skip to content

Add warmstart, model update, update docs - #411

Merged
rapids-bot[bot] merged 4 commits into
NVIDIA:branch-25.10from
Iroy30:add_warmstart_and_model_update
Oct 1, 2025
Merged

Add warmstart, model update, update docs#411
rapids-bot[bot] merged 4 commits into
NVIDIA:branch-25.10from
Iroy30:add_warmstart_and_model_update

Conversation

@Iroy30

@Iroy30 Iroy30 commented Sep 23, 2025

Copy link
Copy Markdown
Member

Description

Added:

  • updateConstraint() - Updates constraint coefficients and rhs
  • updateObjective() - Updates objective coefficients, constant and sense
  • get_pdlp_warm_start() - Retrieves warmstart data
  • relax() - to convert MIP to LP
  • getVariable and getConstraint to get vars and constrs by name or index
  • Updated docs and added tests
  • Fixed Objective expression bug when adding duplicate variables

Issue

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

@Iroy30
Iroy30 requested a review from a team as a code owner September 23, 2025 05:00
@Iroy30
Iroy30 requested a review from tmckayus September 23, 2025 05:00
@copy-pr-bot

copy-pr-bot Bot commented Sep 23, 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.

@ramakrishnap-nv ramakrishnap-nv added doc Improvements or additions to documentation non-breaking Introduces a non-breaking change labels Sep 23, 2025
@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/ok to test

@copy-pr-bot

copy-pr-bot Bot commented Sep 23, 2025

Copy link
Copy Markdown

/ok to test

@rgsl888prabhu, there was an error processing your request: E1

See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/1/

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/ok to test 763ad06

@ramakrishnap-nv ramakrishnap-nv left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you @Iroy30 for adding missing functions, please also add an example under example.rst on how to use warm start

@anandhkb anandhkb added this to the 25.10 milestone Sep 24, 2025
@Iroy30

Iroy30 commented Sep 29, 2025

Copy link
Copy Markdown
Member Author

/ok to test 744600d

@Iroy30

Iroy30 commented Sep 29, 2025

Copy link
Copy Markdown
Member Author

/ok to test b807173

Comment thread docs/cuopt/source/cuopt-python/lp-milp/lp-milp-api.rst
@Iroy30

Iroy30 commented Sep 29, 2025

Copy link
Copy Markdown
Member Author

/ok to test a26cfd4

@Iroy30
Iroy30 force-pushed the add_warmstart_and_model_update branch from 0cb2a6c to ddd2fcd Compare September 29, 2025 22:07
@Iroy30

Iroy30 commented Sep 29, 2025

Copy link
Copy Markdown
Member Author

/ok to test ddd2fcd

@ramakrishnap-nv ramakrishnap-nv left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Minor change, rest looks good. Awesome work @Iroy30.

Comment thread docs/cuopt/source/cuopt-python/lp-milp/lp-milp-examples.rst Outdated
@anandhkb

Copy link
Copy Markdown
Contributor

/ok to test

@copy-pr-bot

copy-pr-bot Bot commented Sep 30, 2025

Copy link
Copy Markdown

/ok to test

@anandhkb, there was an error processing your request: E1

See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/1/

@Iroy30

Iroy30 commented Oct 1, 2025

Copy link
Copy Markdown
Member Author

/ok to test 8e1953d

@Iroy30

Iroy30 commented Oct 1, 2025

Copy link
Copy Markdown
Member Author

/merge

@rapids-bot
rapids-bot Bot merged commit 6b11a4f into NVIDIA:branch-25.10 Oct 1, 2025
252 of 260 checks passed
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
Added:

- updateConstraint() - Updates constraint coefficients and rhs
- updateObjective() - Updates  objective coefficients, constant and sense
- get_pdlp_warm_start() - Retrieves warmstart data
- relax() -  to convert MIP to LP
- getVariable and getConstraint to get vars and constrs by name or index
- Updated docs and added tests
- Fixed Objective expression bug when adding duplicate variables

## Issue

Authors:
  - Ishika Roy (https://github.com/Iroy30)

Approvers:
  - Cindy Wilkinson (https://github.com/cwilkinson76)
  - Trevor McKay (https://github.com/tmckayus)
  - Ramakrishnap (https://github.com/rgsl888prabhu)

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

Labels

doc Improvements or additions to documentation non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants