use deepcopy on constructors in runner to allow for pop to be used

gidden · Sep 6, 2017 · 503ce72 · 503ce72
1 parent 34a598e
commit 503ce72
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/salamanca/models/project.py b/salamanca/models/project.py
@@ -1,5 +1,6 @@
 from __future__ import division
 
+import copy
 import itertools
 import math
 
@@ -350,8 +351,11 @@ def _update(self, t, df):
     def make_model(self, t1, t2):
         self.solve_t = t2
         ndf, sdf = self._data(t1, t2)
-        self.model = self.Model(ndf, sdf, **self.constructor_kwargs)
-        self.model.construct(**self.model_kwargs)
+        # explicitly copies so pop can be used
+        kwargs = copy.deepcopy(self.constructor_kwargs)
+        self.model = self.Model(ndf, sdf, **kwargs)
+        kwargs = copy.deepcopy(self.model_kwargs)
+        self.model.construct(**kwargs)
         return self
 
     def solve(self, *args, **kwargs):