Skip to content

Update stubs with node selection methods #1101

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.

Sign up for GitHub

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 5 commits into
base: master
Choose a base branch
from
Open

Update stubs with node selection methods #1101

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
80c7a56
Add new matmul definition
jonathanberthias Oct 26, 2025
c05ba96
Remove remaining __reduce_cython__
jonathanberthias Oct 30, 2025
34a0bd7
Merge branch 'master' into stubs-update
Joao-Dionisio Nov 6, 2025
73261a7
Add stubs for node selection methods
jonathanberthias Nov 8, 2025
914be61
Merge branch 'master' into stubs-update
Joao-Dionisio Nov 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 7 additions & 4 deletions src/pyscipopt/scip.pyi
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -236,12 +236,10 @@ class Expr:
def __next__(self): ...
def __pow__(self, other): ...
def __radd__(self, other): ...
def __reduce_cython__(self, *args, **kwargs): ...
Copy link
Member

@Joao-Dionisio Joao-Dionisio Nov 10, 2025

Choose a reason for hiding this comment

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

what were these and why don't we need them anymore?

Copy link
Contributor Author

@jonathanberthias jonathanberthias Nov 11, 2025

Choose a reason for hiding this comment

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

These are symbols generated by Cython that exist at runtime so stubgen picked them up, but I don't think they are useful to have in the stubs, as I wouldn't expect anyone to try to use them. But maybe I'm wrong, in which case we can add them back 🤷

I removed most of these symbols in #1073, these are a few leftovers I missed back then

def __rmul__(self, other): ...
def __rpow__(self, other): ...
def __rsub__(self, other): ...
def __rtruediv__(self, other): ...
def __setstate_cython__(self, *args, **kwargs): ...
def __sub__(self, other): ...
def __truediv__(self, other): ...

Expand Down Expand Up @@ -274,12 +272,10 @@ class GenExpr:
def __neg__(self): ...
def __pow__(self, other): ...
def __radd__(self, other): ...
def __reduce_cython__(self, *args, **kwargs): ...
def __rmul__(self, other): ...
def __rpow__(self, other): ...
def __rsub__(self, other): ...
def __rtruediv__(self, other): ...
def __setstate_cython__(self, *args, **kwargs): ...
def __sub__(self, other): ...
def __truediv__(self, other): ...

Expand Down Expand Up @@ -357,6 +353,7 @@ class MatrixExpr(numpy.ndarray):
def __ge__(self, other: object) -> bool: ...
def __iadd__(self, other): ...
def __le__(self, other: object) -> bool: ...
def __matmul__(self, *args, **kwargs): ...
def __mul__(self, other): ...
def __pow__(self, other): ...
def __radd__(self, other): ...
Expand Down Expand Up @@ -536,6 +533,7 @@ class Model:
def getCurrentNode(self, *args, **kwargs): ...
def getCutEfficacy(self, *args, **kwargs): ...
def getCutLPSolCutoffDistance(self, *args, **kwargs): ...
def getCutoffbound(self, *args, **kwargs): ...
def getDepth(self, *args, **kwargs): ...
def getDualMultiplier(self, *args, **kwargs): ...
def getDualSolVal(self, *args, **kwargs): ...
Expand All @@ -559,7 +557,9 @@ class Model:
def getLhs(self, *args, **kwargs): ...
def getLinearConsIndicator(self, *args, **kwargs): ...
def getLocalEstimate(self, *args, **kwargs): ...
def getLowerbound(self, *args, **kwargs): ...
def getMajorVersion(self, *args, **kwargs): ...
def getMaxDepth(self, *args, **kwargs): ...
def getMinorVersion(self, *args, **kwargs): ...
def getNBestSolsFound(self, *args, **kwargs): ...
def getNBinVars(self, *args, **kwargs): ...
Expand All @@ -581,12 +581,14 @@ class Model:
def getNLeaves(self, *args, **kwargs): ...
def getNLimSolsFound(self, *args, **kwargs): ...
def getNNlRows(self, *args, **kwargs): ...
def getNNodeLPIterations(self, *args, **kwargs): ...
def getNNodes(self, *args, **kwargs): ...
def getNReaders(self, *args, **kwargs): ...
def getNSepaRounds(self, *args, **kwargs): ...
def getNSiblings(self, *args, **kwargs): ...
def getNSols(self, *args, **kwargs): ...
def getNSolsFound(self, *args, **kwargs): ...
def getNStrongbranchLPIterations(self, *args, **kwargs): ...
def getNTotalNodes(self, *args, **kwargs): ...
def getNVars(self, *args, **kwargs): ...
def getNVarsAnd(self, *args, **kwargs): ...
Expand All @@ -602,6 +604,7 @@ class Model:
def getOpenNodes(self, *args, **kwargs): ...
def getParam(self, *args, **kwargs): ...
def getParams(self, *args, **kwargs): ...
def getPlungeDepth(self, *args, **kwargs): ...
def getPresolvingTime(self, *args, **kwargs): ...
def getPrimalRay(self, *args, **kwargs): ...
def getPrimalRayVal(self, *args, **kwargs): ...
Expand Down
Loading