Drop Variable.create via __cinit__

Replaces the static Variable.create(SCIP_VAR*) method with direct instantiation using the Variable constructor and casting pointers to size_t. Removes the create() method from the Variable class definition and updates all usages accordingly for consistency and clarity.

Author: Zeroto521

src/pyscipopt/propagator.pxi

@@ -148,7 +148,7 @@ cdef SCIP_RETCODE PyPropResProp (SCIP* scip, SCIP_PROP* prop, SCIP_VAR* infervar
                                  SCIP_BOUNDTYPE boundtype, SCIP_BDCHGIDX* bdchgidx, SCIP_Real relaxedbd, SCIP_RESULT* result) noexcept with gil:
     cdef SCIP_PROPDATA* propdata
     propdata = SCIPpropGetData(prop)
-    confvar = Variable.create(infervar)
+    confvar = Variable(<size_t>infervar)
 
     #TODO: parse bdchgidx?
 

src/pyscipopt/reader.pxi

@@ -52,11 +52,11 @@ cdef SCIP_RETCODE PyReaderWrite (SCIP* scip, SCIP_READER* reader, FILE* file,
 
     PyFile = os.fdopen(fd, "w", closefd=False)
     PyName = name.decode('utf-8')
-    PyBinVars = [Variable.create(vars[i]) for i in range(nbinvars)]
-    PyIntVars = [Variable.create(vars[i]) for i in range(nbinvars, nintvars)]
-    PyImplVars = [Variable.create(vars[i]) for i in range(nintvars, nimplvars)]
-    PyContVars = [Variable.create(vars[i]) for i in range(nimplvars, ncontvars)]
-    PyFixedVars = [Variable.create(fixedvars[i]) for i in range(nfixedvars)]
+    PyBinVars = [Variable(<size_t>vars[i]) for i in range(nbinvars)]
+    PyIntVars = [Variable(<size_t>vars[i]) for i in range(nbinvars, nintvars)]
+    PyImplVars = [Variable(<size_t>vars[i]) for i in range(nintvars, nimplvars)]
+    PyContVars = [Variable(<size_t>vars[i]) for i in range(nimplvars, ncontvars)]
+    PyFixedVars = [Variable(<size_t>fixedvars[i]) for i in range(nfixedvars)]
     PyConss = [Constraint.create(conss[i]) for i in range(nconss)]
     PyReader = <Reader>readerdata
     #TODO: provide rational objoffsetexact and objscaleexact

src/pyscipopt/scip.pxd

@@ -2188,9 +2188,6 @@ cdef class Variable:
     # can be used to store problem data
     cdef public object data
 
-    @staticmethod
-    cdef create(SCIP_VAR* scipvar)
-
 cdef class Constraint:
     cdef SCIP_CONS* scip_cons
     # can be used to store problem data

src/pyscipopt/scip.pxi

@@ -445,8 +445,7 @@ cdef class Event:
         Variable
 
         """
-        cdef SCIP_VAR* var = SCIPeventGetVar(self.event)
-        return Variable.create(var)
+        return Variable(<size_t>SCIPeventGetVar(self.event))
 
     def getNode(self):
         """
@@ -567,7 +566,7 @@ cdef class Column:
 
         """
         cdef SCIP_VAR* var = SCIPcolGetVar(self.scip_col)
-        return Variable.create(var)
+        return Variable(<size_t>var)
 
     def getPrimsol(self):
         """
@@ -1020,7 +1019,7 @@ cdef class NLRow:
         cdef SCIP_Real* lincoefs = SCIPnlrowGetLinearCoefs(self.scip_nlrow)
         cdef int nlinvars = SCIPnlrowGetNLinearVars(self.scip_nlrow)
         cdef int i
-        return [(Variable.create(linvars[i]), lincoefs[i]) for i in range(nlinvars)]
+        return [(Variable(<size_t>linvars[i]), lincoefs[i]) for i in range(nlinvars)]
 
     def getLhs(self):
         """
@@ -1221,7 +1220,7 @@ cdef class BoundChange:
         Variable
 
         """
-        return Variable.create(SCIPboundchgGetVar(self.scip_boundchg))
+        return Variable(<size_t>SCIPboundchgGetVar(self.scip_boundchg))
 
     def getBoundchgtype(self):
         """
@@ -1489,7 +1488,7 @@ cdef class Node:
         SCIPnodeGetParentBranchings(self.scip_node, branchvars, branchbounds,
                                     boundtypes, &nbranchvars, nbranchvars)
 
-        py_variables = [Variable.create(branchvars[i]) for i in range(nbranchvars)]
+        py_variables = [Variable(<size_t>branchvars[i]) for i in range(nbranchvars)]
         py_branchbounds = [branchbounds[i] for i in range(nbranchvars)]
         py_boundtypes = [boundtypes[i] for i in range(nbranchvars)]
         free(boundtypes)
@@ -1538,28 +1537,8 @@ cdef class Node:
 
 cdef class Variable:
 
-    @staticmethod
-    cdef create(SCIP_VAR* scip_var):
-        """
-        Main method for creating a Variable class. Is used instead of __init__.
-
-        Parameters
-        ----------
-        scip_var : SCIP_VAR*
-            A pointer to the SCIP_VAR
-
-        Returns
-        -------
-        var : Variable
-            The Python representative of the SCIP_VAR
-
-        """
-        if scip_var == NULL:
-            raise Warning("cannot create Variable with SCIP_VAR* == NULL")
-
-        var = Variable()
-        var.scip_var = scip_var
-        return var
+    def __cinit__(self, size_t scip_var):
+        self.scip_var = <SCIP_VAR*>scip_var
 
     @property
     def name(self):
@@ -2034,6 +2013,7 @@ cdef class Variable:
         """
         return SCIPvarGetNBranchingsCurrentRun(self.scip_var, branchdir)
 
+
 class MatrixVariable(MatrixExpr):
 
     def vtype(self):
@@ -4292,7 +4272,7 @@ cdef class Model:
         else:
             PY_SCIP_CALL(SCIPaddVar(self._scip, scip_var))
 
-        pyVar = Variable.create(scip_var)
+        pyVar = Variable(<size_t>scip_var)
 
         # store variable in the model to avoid creating new python variable objects in getVars()
         assert not pyVar.ptr() in self._modelvars
@@ -4428,7 +4408,7 @@ cdef class Model:
         cdef SCIP_VAR* _tvar
         PY_SCIP_CALL(SCIPgetTransformedVar(self._scip, var.scip_var, &_tvar))
 
-        return Variable.create(_tvar)
+        return Variable(<size_t>_tvar)
 
     def addVarLocks(self, Variable var, int nlocksdown, int nlocksup):
         """
@@ -4791,7 +4771,7 @@ cdef class Model:
                 vars.append(self._modelvars[ptr])
             else:
                 # create a new variable
-                var = Variable.create(_vars[i])
+                var = Variable(<size_t>_vars[i])
                 assert var.ptr() == ptr
                 self._modelvars[ptr] = var
                 vars.append(var)
@@ -6542,7 +6522,7 @@ cdef class Model:
                 vars.append(self._modelvars[ptr])
             else:
                 # create a new variable
-                var = Variable.create(_vars[i])
+                var = Variable(<size_t>_vars[i])
                 assert var.ptr() == ptr
                 self._modelvars[ptr] = var
                 vars.append(var)
@@ -6631,7 +6611,7 @@ cdef class Model:
                 vars.append(self._modelvars[ptr])
             else:
                 # create a new variable
-                var = Variable.create(_vars[i])
+                var = Variable(<size_t>_vars[i])
                 assert var.ptr() == ptr
                 self._modelvars[ptr] = var
                 vars.append(var)
@@ -6661,7 +6641,7 @@ cdef class Model:
         # check whether the corresponding variable exists already
         if ptr not in self._modelvars:
             # create a new variable
-            resultant = Variable.create(_resultant)
+            resultant = Variable(<size_t>_resultant)
             assert resultant.ptr() == ptr
             self._modelvars[ptr] = resultant
         else:
@@ -7565,7 +7545,7 @@ cdef class Model:
 
         """
         cdef SCIP_VAR* var = SCIPgetSlackVarIndicator(cons.scip_cons)
-        return Variable.create(var)
+        return Variable(<size_t>var)
 
     def addPyCons(self, Constraint cons):
         """
@@ -8188,15 +8168,15 @@ cdef class Model:
         quadterms  = []
 
         for termidx in range(nlinvars):
-            var = Variable.create(SCIPgetVarExprVar(linexprs[termidx]))
+            var = Variable(<size_t>SCIPgetVarExprVar(linexprs[termidx]))
             linterms.append((var, lincoefs[termidx]))
 
         for termidx in range(nbilinterms):
             SCIPexprGetQuadraticBilinTerm(expr, termidx, &bilinterm1, &bilinterm2, &bilincoef, NULL, NULL)
             scipvar1 = SCIPgetVarExprVar(bilinterm1)
             scipvar2 = SCIPgetVarExprVar(bilinterm2)
-            var1 = Variable.create(scipvar1)
-            var2 = Variable.create(scipvar2)
+            var1 = Variable(<size_t>scipvar1)
+            var2 = Variable(<size_t>scipvar2)
             if scipvar1 != scipvar2:
                 bilinterms.append((var1,var2,bilincoef))
             else:
@@ -8206,7 +8186,7 @@ cdef class Model:
             SCIPexprGetQuadraticQuadTerm(expr, termidx, NULL, &lincoef, &sqrcoef, NULL, NULL, &sqrexpr)
             if sqrexpr == NULL:
                 continue
-            var = Variable.create(SCIPgetVarExprVar(sqrexpr))
+            var = Variable(<size_t>SCIPgetVarExprVar(sqrexpr))
             quadterms.append((var,sqrcoef,lincoef))
 
         return (bilinterms, quadterms, linterms)
@@ -8899,7 +8879,7 @@ cdef class Model:
         if _mappedvar == NULL:
             mappedvar = None
         else:
-            mappedvar = Variable.create(_mappedvar)
+            mappedvar = Variable(<size_t>_mappedvar)
 
         return mappedvar
 
@@ -8928,7 +8908,7 @@ cdef class Model:
             _benders = benders._benders
 
         _auxvar = SCIPbendersGetAuxiliaryVar(_benders, probnumber)
-        auxvar = Variable.create(_auxvar)
+        auxvar = Variable(<size_t>_auxvar)
 
         return auxvar
 
@@ -9723,7 +9703,7 @@ cdef class Model:
         PY_SCIP_CALL(SCIPgetLPBranchCands(self._scip, &lpcands, &lpcandssol, &lpcandsfrac,
                                           &nlpcands, &npriolpcands, &nfracimplvars))
 
-        return ([Variable.create(lpcands[i]) for i in range(nlpcands)], [lpcandssol[i] for i in range(nlpcands)],
+        return ([Variable(<size_t>lpcands[i]) for i in range(nlpcands)], [lpcandssol[i] for i in range(nlpcands)],
                 [lpcandsfrac[i] for i in range(nlpcands)], nlpcands, npriolpcands, nfracimplvars)
 
     def getNLPBranchCands(self):
@@ -9760,7 +9740,7 @@ cdef class Model:
 
         PY_SCIP_CALL(SCIPgetPseudoBranchCands(self._scip, &pseudocands, &npseudocands, &npriopseudocands))
 
-        return ([Variable.create(pseudocands[i]) for i in range(npseudocands)], npseudocands, npriopseudocands)
+        return ([Variable(<size_t>pseudocands[i]) for i in range(npseudocands)], npseudocands, npriopseudocands)
 
     def branchVar(self, Variable variable):
         """