From 2dcefe1a68c3b2b5c5806caad38416da0f660dda Mon Sep 17 00:00:00 2001 From: Ronan Fleming Date: Thu, 7 Nov 2024 16:08:14 +0000 Subject: [PATCH] small changes --- .../cardOpt/sparseLP/optimizeCardinality.m | 8 +- .../solvers/getSetSolver/changeCobraSolver.m | 4 +- .../SRR8994378_S47D_flux_shifts.xls | Bin 2110464 -> 2110464 bytes .../nonFluxReactionsnonFluxReactionsWT.xls | Bin 45056 -> 45056 bytes .../_rate_limitting_reactionsWT.xls | Bin 302592 -> 302592 bytes .../testEFMviz/testEFMSubmodel.xml | 1002 ++++++++++++++--- .../testEFMviz/testEFMSubmodel_woUbMets.xml | 810 ++++++++++--- 7 files changed, 1518 insertions(+), 306 deletions(-) diff --git a/src/base/solvers/cardOpt/sparseLP/optimizeCardinality.m b/src/base/solvers/cardOpt/sparseLP/optimizeCardinality.m index 2db3d0850f..640d7a49f5 100644 --- a/src/base/solvers/cardOpt/sparseLP/optimizeCardinality.m +++ b/src/base/solvers/cardOpt/sparseLP/optimizeCardinality.m @@ -195,8 +195,8 @@ ltp=length(problem.p); ltq=length(problem.q); ltr=length(problem.r); - if ltp==1 - if ~islogical(problem.p) + if ltp==1 && problem.p~=0 + if length(problem.p)~=1 && ~islogical(problem.p) solution.stat = -1; error('Error: p should be a logical vector'); end @@ -212,7 +212,7 @@ solution.stat = -1; error('Error: the size/location of vector y is not defined'); else - if ~islogical(problem.q) + if length(problem.q)~=1 && ~islogical(problem.q) && problem.q~=0 solution.stat = -1; error('Error: q should be a logical vector'); end @@ -222,7 +222,7 @@ solution.stat = -1; error('Error: the size of vector z is not defined'); else - if ~islogical(problem.r) + if length(problem.r)~=1 && ~islogical(problem.r) solution.stat = -1; error('Error: r should be a logical vector'); end diff --git a/src/base/solvers/getSetSolver/changeCobraSolver.m b/src/base/solvers/getSetSolver/changeCobraSolver.m index 24e4863f4c..7e76253c42 100644 --- a/src/base/solvers/getSetSolver/changeCobraSolver.m +++ b/src/base/solvers/getSetSolver/changeCobraSolver.m @@ -3,7 +3,7 @@ % % USAGE: % -% [solverOK, solverInstalled] = changeCobraSolver(solverName, solverType, printLevel, validationLevel) +% [solverOK, solverInstalled] = changeCobraSolver(solverName, problemType, printLevel, validationLevel) % % INPUTS: % solverName: Solver name @@ -538,7 +538,7 @@ end try % Skip the CLP solver until further developments - if ~strcmp(solverType, 'CLP') + if ~strcmp(problemType, 'CLP') %This is the code that actually tests if a solver is working if validationLevel>1 %display progress diff --git a/test/verifiedTests/analysis/testCalculateFluxShifts/resultsPostOptimization/fluxShifts/SRR8994378_S47D_flux_shifts.xls b/test/verifiedTests/analysis/testCalculateFluxShifts/resultsPostOptimization/fluxShifts/SRR8994378_S47D_flux_shifts.xls index 77edf39d0dbb88116dae96a8334a7b436bfa71e8..7dbff65874133f47421293e5598d668ae728e7da 100644 GIT binary patch delta 148 zcmWm2xe-7>06@`kt@~cqO#>QH)y@c{N|9-aE_}gDo_@5UaY2CnXwjj^fDsesTZ!deKQiha`Tzg` delta 148 zcmWm2$q_&>06@{PMeMuSX+TGc$&7;v^x>u@y6_#|<>^No8doW-bCbIghY|)%Sg_&X le-{!Sd;|y)Ax44}8FCaTQK5!HgBBfn3>YzCzLi+s^#eWx9uEKj diff --git a/test/verifiedTests/analysis/testFilterNonFluxReactions/resultsPostOptimization/nonFluxReactionsnonFluxReactionsWT.xls b/test/verifiedTests/analysis/testFilterNonFluxReactions/resultsPostOptimization/nonFluxReactionsnonFluxReactionsWT.xls index 16b5a794b81145580ec2b5575841f1c0d301aa3d..c48f88a335737f8b7483fb943c60e5d61f7f91f0 100644 GIT binary patch delta 17 ZcmZp8z|`=7X@e6R(-Eo7E^K>N0sufu2LAv6 delta 17 YcmZp8z|`=7X@e6RQxflH7q&es0XYt61jNkSUD#OmOa}nS(+bl7 delta 33 lcmZqJBh;`*XoC|QQ(R%Q3tPJj8zT@i0WtG-7dDnX(*d@t3IqTE diff --git a/test/verifiedTests/visualization/testEFMviz/testEFMSubmodel.xml b/test/verifiedTests/visualization/testEFMviz/testEFMSubmodel.xml index dfae7967cb..a9f5195922 100644 --- a/test/verifiedTests/visualization/testEFMviz/testEFMSubmodel.xml +++ b/test/verifiedTests/visualization/testEFMviz/testEFMSubmodel.xml @@ -283,7 +283,7 @@ - + @@ -352,7 +352,7 @@ - + @@ -379,7 +379,7 @@ - + @@ -408,9 +408,9 @@ - + - + @@ -578,14 +578,14 @@ - - - - - - + + + + + + - + @@ -610,14 +610,14 @@ - - - - - - + + + + + + - + @@ -642,14 +642,14 @@ - - - - - - + + + + + + - + @@ -674,13 +674,13 @@ - - - + + + - - + + @@ -715,7 +715,7 @@ - + @@ -747,7 +747,7 @@ - + @@ -772,7 +772,7 @@ - + @@ -797,7 +797,7 @@ - + @@ -820,16 +820,16 @@ - - - - + + + + - - + + - - + + @@ -951,9 +951,9 @@ - + - + @@ -1109,12 +1109,12 @@ - - - + + + - + @@ -1227,9 +1227,9 @@ - - - + + + @@ -1241,9 +1241,9 @@ - - - + + + @@ -1315,9 +1315,9 @@ - - - + + + @@ -1477,9 +1477,9 @@ - - - + + + @@ -1514,9 +1514,9 @@ - - - + + + @@ -1554,9 +1554,9 @@ - - - + + + @@ -1590,9 +1590,9 @@ - - - + + + @@ -1762,9 +1762,9 @@ - - - + + + @@ -1797,9 +1797,9 @@ - - - + + + @@ -1837,9 +1837,9 @@ - - - + + + @@ -1892,9 +1892,9 @@ - - - + + + @@ -1932,9 +1932,9 @@ - - - + + + @@ -2009,9 +2009,9 @@ - - - + + + @@ -2104,9 +2104,9 @@ - - - + + + @@ -2173,11 +2173,11 @@ - + - - - + + + @@ -2350,9 +2350,9 @@ - - - + + + @@ -2388,9 +2388,9 @@ - - - + + + @@ -2426,9 +2426,9 @@ - - - + + + @@ -2462,11 +2462,11 @@ - + - - - + + + @@ -2531,7 +2531,7 @@ - + @@ -2560,62 +2560,734 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/test/verifiedTests/visualization/testEFMviz/testEFMSubmodel_woUbMets.xml b/test/verifiedTests/visualization/testEFMviz/testEFMSubmodel_woUbMets.xml index 8226adae4c..5b4f9a164b 100644 --- a/test/verifiedTests/visualization/testEFMviz/testEFMSubmodel_woUbMets.xml +++ b/test/verifiedTests/visualization/testEFMviz/testEFMSubmodel_woUbMets.xml @@ -283,7 +283,7 @@ - + @@ -492,14 +492,14 @@ - - - - - - + + + + + + - + @@ -524,14 +524,14 @@ - - - - - - + + + + + + - + @@ -556,14 +556,14 @@ - - - - - - + + + + + + - + @@ -588,13 +588,13 @@ - - - + + + - - + + @@ -629,7 +629,7 @@ - + @@ -661,7 +661,7 @@ - + @@ -686,7 +686,7 @@ - + @@ -711,7 +711,7 @@ - + @@ -734,9 +734,9 @@ - + - + @@ -863,9 +863,9 @@ - - - + + + @@ -877,9 +877,9 @@ - - - + + + @@ -945,9 +945,9 @@ - - - + + + @@ -1085,9 +1085,9 @@ - - - + + + @@ -1122,9 +1122,9 @@ - - - + + + @@ -1162,9 +1162,9 @@ - - - + + + @@ -1198,9 +1198,9 @@ - - - + + + @@ -1356,9 +1356,9 @@ - - - + + + @@ -1390,9 +1390,9 @@ - - - + + + @@ -1427,9 +1427,9 @@ - - - + + + @@ -1481,9 +1481,9 @@ - - - + + + @@ -1519,9 +1519,9 @@ - - - + + + @@ -1557,9 +1557,9 @@ - - - + + + @@ -1652,9 +1652,9 @@ - - - + + + @@ -1717,11 +1717,11 @@ - + - - - + + + @@ -1838,9 +1838,9 @@ - - - + + + @@ -1876,9 +1876,9 @@ - - - + + + @@ -1910,11 +1910,11 @@ - + - - - + + + @@ -1977,7 +1977,7 @@ - + @@ -2006,51 +2006,591 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +