Skip to content
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

Added function to the C API: #620

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mapgccv
Copy link

@mapgccv mapgccv commented Sep 19, 2023

Hi team, thanks for the wonderful work you're doing.
While developing a solver driver using CBC through the C API, I came across a few functionalities that I just could not find. I added them to the C API (as you see in the commit), although I'm not sure if there were other (preferred) ways to get/set the same information.

Osi_setColSolution
Osi_setRowPrice
Osi_setBasisStatus
Osi_getBasisStatus
@CLAassistant
Copy link

CLAassistant commented Sep 19, 2023

CLA assistant check
All committers have signed the CLA.

@svigerske
Copy link
Member

You've added them to the OsiCuts_Xyz functions in the hpp file.
Please move them up into the section

/** \name OsiSolverInterface related routines (used in callbacks) */

Also the location in the cpp file does not seem correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants