fix: recursion problem in glpk_interface (#59)

_remove_variables would call self.variables for models with more than 350 variables.
opencobra · Jan 20, 2017 · ff08502 · ff08502
1 parent 0d416e8
commit ff08502
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/optlang/glpk_interface.py b/optlang/glpk_interface.py
@@ -677,8 +677,8 @@ def _remove_variables(self, variables):
         if len(variables) > 0:
             if len(variables) > 350:
                 delete_indices = [variable._index - 1 for variable in variables]
-                keep_indices = [i for i in range(0, len(self.variables)) if i not in delete_indices]
-                self._variables = self.variables.fromkeys(keep_indices)
+                keep_indices = [i for i in range(0, len(self._variables)) if i not in delete_indices]
+                self._variables = self._variables.fromkeys(keep_indices)
             else:
                 for variable in variables:
                     del self._variables[variable.name]