Skip to content

Conversation

@Iroy30
Copy link
Member

@Iroy30 Iroy30 commented Sep 23, 2025

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 link

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

@rgsl888prabhu rgsl888prabhu added doc Improvements or additions to documentation non-breaking Introduces a non-breaking change labels Sep 23, 2025
@rgsl888prabhu
Copy link
Collaborator

/ok to test

@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 23, 2025

/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/

@rgsl888prabhu
Copy link
Collaborator

/ok to test 763ad06

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.

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
Copy link
Member Author

Iroy30 commented Sep 29, 2025

/ok to test 744600d

@Iroy30
Copy link
Member Author

Iroy30 commented Sep 29, 2025

/ok to test b807173

@Iroy30
Copy link
Member Author

Iroy30 commented Sep 29, 2025

/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
Copy link
Member Author

Iroy30 commented Sep 29, 2025

/ok to test ddd2fcd

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.

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

@anandhkb
Copy link
Contributor

/ok to test

@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 30, 2025

/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
Copy link
Member Author

Iroy30 commented Oct 1, 2025

/ok to test 8e1953d

@Iroy30
Copy link
Member Author

Iroy30 commented Oct 1, 2025

/merge

@rapids-bot rapids-bot bot merged commit 6b11a4f into NVIDIA:branch-25.10 Oct 1, 2025
252 of 260 checks passed
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