From 0c9f0f6bd7467e8a45bf64ab2dd1f36dfbfa569c Mon Sep 17 00:00:00 2001 From: Andrew Freiburger Date: Wed, 26 Aug 2026 14:49:02 -0500 Subject: [PATCH] Sync modelseedpy/fbapkg/problemreplicationpkg.py from freiburgermsu fork File-level sync to the state of this file in freiburgermsu/ModelSEEDpy@971df5d, submitted as an individual PR so each module's divergence can be reviewed independently. Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_017rcBA87xipNeaukuFW8N2G --- modelseedpy/fbapkg/problemreplicationpkg.py | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/modelseedpy/fbapkg/problemreplicationpkg.py b/modelseedpy/fbapkg/problemreplicationpkg.py index 71e81244..5ffca6e7 100644 --- a/modelseedpy/fbapkg/problemreplicationpkg.py +++ b/modelseedpy/fbapkg/problemreplicationpkg.py @@ -2,12 +2,10 @@ from __future__ import absolute_import -import logging -from optlang import Variable -from optlang.symbolics import Zero, add + from modelseedpy.fbapkg.basefbapkg import BaseFBAPkg -from modelseedpy.fbapkg.revbinpkg import RevBinPkg -from modelseedpy.fbapkg.totalfluxpkg import TotalFluxPkg +from optlang import Variable +import logging # Base class for FBA packages class ProblemReplicationPkg(BaseFBAPkg): @@ -22,12 +20,12 @@ def build_package(self, parameters): shared_var_hash = {} for pkg in self.parameters["shared_variable_packages"]: fbapkg = self.modelutl.pkgmgr.getpkg(pkg) - for type in self.parameters["shared_variable_packages"][pkg]: - if type in fbapkg.variables: - for objid in fbapkg.variables[type]: + for obj_type in self.parameters["shared_variable_packages"][pkg]: + if obj_type in pkg.variables: + for objid in pkg.variables[obj_type]: shared_var_hash[ - fbapkg.variables[type][objid].name - ] = fbapkg.variables[type][objid] + pkg.variables[obj_type][objid].name + ] = pkg.variables[obj_type][objid] # Now copying over variables and constraints from other models and replacing shared variables count = 0 for othermdl in self.parameters["models"]: