Skip to content

Commit f352c30

Browse files
committed
fix test_pricer bug
1 parent 703fd34 commit f352c30

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

tests/test_pricer.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -42,16 +42,13 @@ def pricerredcost(self):
4242
assert type(self.model.getNSolsFound()) == int
4343
assert type(self.model.getNBestSolsFound()) == int
4444
assert self.model.getNBestSolsFound() <= self.model.getNSolsFound()
45-
46-
self.model.data["nSols"] = self.model.getNSolsFound()
4745

4846
# Adding the column to the master problem (model.LT because of numerics)
4947
if self.model.isLT(objval, 0):
5048
currentNumVar = len(self.data['var'])
5149

5250
# Creating new var; must set pricedVar to True
5351
newVar = self.model.addVar("NewPattern_" + str(currentNumVar), vtype = "C", obj = 1.0, pricedVar = True)
54-
5552
# Adding the new variable to the constraints of the master problem
5653
newPattern = []
5754
for i, c in enumerate(self.data['cons']):
@@ -87,7 +84,6 @@ def test_cuttingstock():
8784

8885
s.setPresolve(0)
8986
s.data = {}
90-
s.data["nSols"] = 0
9187

9288
# creating a pricer
9389
pricer = CutPricer()
@@ -164,7 +160,7 @@ def test_cuttingstock():
164160

165161
assert s.getObjVal() == 452.25
166162
assert type(s.getNSols()) == int
167-
assert s.getNSols() == s.data["nSols"]
163+
assert s.getNSols() == s.getNSolsFound()
168164

169165
# Testing freeTransform
170166
s.freeTransform()
@@ -189,7 +185,6 @@ def test_deactivate_pricer():
189185

190186
s.setPresolve(0)
191187
s.data = {}
192-
s.data["nSols"] = 0
193188

194189
# creating a pricer
195190
pricer = CutPricer()

0 commit comments

Comments
 (0)