remove addOrigVarsConssObjectiveFrom

Author: Light1Lee

CHANGELOG.md

@@ -23,7 +23,7 @@
 - Added additional tests to test_nodesel, test_heur, and test_strong_branching
 - Migrated documentation to Readthedocs
 - `attachEventHandlerCallback` method to Model for a more ergonomic way to attach event handlers
-- Added Model method: optimizeNogil, addOrigVarsConssObjectiveFrom
+- Added Model method: optimizeNogil
 - Added Solution method: getSolOrigin, retransform, translate
 - Added SCIP.pxd: SCIP_SOLORIGIN, SCIPcopyOrigVars, SCIPcopyOrigConss, SCIPsolve nogil, SCIPretransformSol, SCIPtranslateSubSol, SCIPsolGetOrigin, SCIPhashmapCreate, SCIPhashmapFree
 - Added additional tests to test_multi_threads, test_solution, and test_copy

src/pyscipopt/scip.pxi

@@ -2094,29 +2094,6 @@ cdef class Model:
         n = str_conversion(problemName)
         PY_SCIP_CALL(SCIPcreateProbBasic(self._scip, n))
 
-    def addOrigVarsConssObjectiveFrom(self, Model source):
-        """
-        add original variables and constraints from source model.
-
-        Parameters
-        ----------
-        source : Model
-            source model copy original variables and constraints to target(self) model
-        """
-        cdef SCIP_Bool valid
-        cdef SCIP_HASHMAP* localvarmap
-        cdef SCIP_HASHMAP* localconsmap
-
-        PY_SCIP_CALL( SCIPhashmapCreate(&localvarmap, SCIPblkmem(self._scip), SCIPgetNVars(source._scip)) )
-        PY_SCIP_CALL( SCIPhashmapCreate(&localconsmap, SCIPblkmem(self._scip), SCIPgetNConss(source._scip)) )
-
-        PY_SCIP_CALL(SCIPcopyOrigVars(source._scip, self._scip, localvarmap, localconsmap, NULL, NULL, 0))
-        PY_SCIP_CALL(SCIPcopyOrigConss(source._scip, self._scip, localvarmap, localconsmap, False, &valid))
-        PY_SCIP_CALL(SCIPsetObjsense(self._scip, SCIPgetObjsense(source._scip)))
-
-        SCIPhashmapFree(&localvarmap)
-        SCIPhashmapFree(&localconsmap)
-
     def freeProb(self):
         """Frees problem and solution process data."""
         PY_SCIP_CALL(SCIPfreeProb(self._scip))

tests/test_copy.py

@@ -20,22 +20,3 @@ def test_copy():
 
     assert s.getObjVal() == s2.getObjVal()
 
-
-def test_addOrigVarsConssObjectiveFrom():
-    m = Model()
-    x = m.addVar("x", vtype = 'B')
-    y = m.addVar("y", vtype = 'B')
-    m.addCons(x + y >= 1)
-    m.addCons(x + y <= 2)
-    m.setObjective(x + y, 'maximize')
-
-    m1 = Model()
-    m1.addOrigVarsConssObjectiveFrom(m)
-
-    m.optimize()
-    m1.optimize()
-
-    assert m.getNVars(transformed=False) == m1.getNVars(transformed=False)
-    assert m.getNConss(transformed=False) == m1.getNConss(transformed=False)
-    assert m.getObjVal() == m1.getObjVal() == 2 
-