Skip to content

Documentation Suggestions for build.rst and lp.pxi#1026

Merged
Joao-Dionisio merged 3 commits into
scipopt:masterfrom
gtjusila:master
Jul 18, 2025
Merged

Documentation Suggestions for build.rst and lp.pxi#1026
Joao-Dionisio merged 3 commits into
scipopt:masterfrom
gtjusila:master

Conversation

@gtjusila

Copy link
Copy Markdown
Contributor

Minor documentation suggestion

  • Added in "build from source" documentation (build.rst) that DYLD_LIBRARY_PATH should be set on MacOS to ensure that PySCIPOPT can find scip dynamic library during runtime
  • Clarified the formatting of entries and entries list for functions addRow, addRows, addCol, and addCols in the LP interface

@Joao-Dionisio

Copy link
Copy Markdown
Member

Thank you @gtjusila! Would you mind updating this line

Please note that some Mac configurations require adding the library installation path to DYLD_LIBRARY_PATH when using a locally installed version of SCIP.

in INSTALL.md, please? Your comment is more informative.

@gtjusila

Copy link
Copy Markdown
Contributor Author

Added the note also to install.md and changed all OS X reference to MacOS for consistency.

@Joao-Dionisio Joao-Dionisio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@Joao-Dionisio Joao-Dionisio merged commit 3544861 into scipopt:master Jul 18, 2025
1 check passed
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.

2 participants