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

Add Cbc_modifyCoefficient to C interface. #656

Closed
wants to merge 1 commit into from

Conversation

jetuk
Copy link

@jetuk jetuk commented Jul 2, 2024

As discussed in #655.

I can do the OSI one too if you like?

@CLAassistant
Copy link

CLAassistant commented Jul 2, 2024

CLA assistant check
All committers have signed the CLA.

@jetuk jetuk force-pushed the c-interface-modify-coefficient branch from 713fd22 to 62509bd Compare July 2, 2024 20:10
@jetuk jetuk changed the title Add Cbc_modifyCoefcient to C interface. Add Cbc_modifyCoefficient to C interface. Jul 2, 2024
@jjhforrest
Copy link
Contributor

Does it help you go faster?

Why not do Osi as well

@jetuk
Copy link
Author

jetuk commented Jul 3, 2024

I can not immediately see how this can be achieved through the OSI. I admit I am not familiar with it though.

@jjhforrest
Copy link
Contributor

Wait to see if anyone wants to do this from Osi C interface. If so, I can help.

@tkralphs
Copy link
Member

I'm not really clear on what Osi C interface we're talking about and why we're waiting for that to be modified first. This looks reasonable to add just as it is.

@jjhforrest
Copy link
Contributor

Modified Cbc_C to add modifyCoefficient. As it calls solver_->modifyCoefficient then Osi needs modifying - done

@jetuk
Copy link
Author

jetuk commented Aug 13, 2024

So, this can be closed?

@jjhforrest
Copy link
Contributor

Yes - if it works for you.

@jetuk jetuk closed this Aug 13, 2024
@jetuk
Copy link
Author

jetuk commented Aug 13, 2024

Thanks for making the change!

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.

4 participants