Skip to content

Fixing typos in scip.pyx file #1

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

Open
wants to merge 160 commits into
base: master
Choose a base branch
from
Open

Fixing typos in scip.pyx file #1

wants to merge 160 commits into from

Conversation

andynuaa
Copy link

@andynuaa andynuaa commented Apr 4, 2019

Hi Maher @stephenjmaher, I appreciate your help in developing the benders related plugins. Follow the suggestion of mattmilten, I now fork the project and propose a pull request to show the differences I made so far in the file scip.pyx.

Stephen J. Maher and others added 30 commits April 9, 2018 20:21
- add functions for adding cuts
- add function to access tableau information
- add test of a separator
close scipopt#158 consistent number check in expr.pxi
…or_separators

improve support cutting planes
ramabile and others added 28 commits October 31, 2018 13:43
explain assertions + .gitignore fix + cosmetics
cf. scipopt#212: right assertion error message
…n-dividing-by-number

upgrade to general expression only when necessary
Bugfix: setObjLimit -> getObjLimit typo
…kx-connected-comps

update networkx call to get connected components
Based on Maher's version, few typos are fixed
@stephenjmaher
Copy link
Owner

There is no need to create a pull request with my fork of the PySCIPOpt. It is best to fork from the SCIP-Interfaces, so that you have an up-to-date version of the code. Then we can look at your changes in you fork.

@andynuaa
Copy link
Author

andynuaa commented Apr 9, 2019

Ok, I will fork the original SCIP-Interface. I am sorry that I haven't noticed your reply as this is the first time I make a pull request and no notification for my account

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.

8 participants