Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changes in result for -d and -g #20

Open
ntromas opened this issue Apr 25, 2021 · 4 comments
Open

Changes in result for -d and -g #20

ntromas opened this issue Apr 25, 2021 · 4 comments

Comments

@ntromas
Copy link

ntromas commented Apr 25, 2021

Hi Daniel,

I am using SMETANA with a small community (models from the last version of carveme, without gapfilling). My output always change for -d which I think is linked to the use of Cplex? What would you suggest to avoid or take into account this variation (e.g different number of interactions)?

I also got different results with -g (using the same input). I tried --molweight just in case but got this error.

cat debug.tsv
community medium key1 key2 data
all complete mip ni 4abz,LalaDgluMdap,acmana,alaala,amp,arg__L,bz,ca2,cgly,cl,cobalt2,cu2,cytd,fe3,frmd,gln__L,gly_asn__L,glyb,glyc3p,hom__L,ile__L,k,lys__L,mg2,mn2,nmn,o2,phe__L,pnto__R,pro__L,quin,ribflv,salchs4fe,so4,thm,tol,tyr__L,uaccg,udcpp,val__L,zn2
all complete mip i 4abz,LalaDgluMdap,acmana,alaala,amp,arg__L,bz,ca2,cgly,cl,cobalt2,cu2,cytd,fe3,frmd,gln__L,gly_asn__L,glyc3p,ile__L,k,lys__L,mg2,mn2,nmn,o2,phe__L,pnto__R,pro__L,quin,ribflv,so4,tol,tyr__L,uaccg,udcpp,val__L,zn2
all complete mro community LalaDgluMdap,acmana,alaala,amp,arg__L,bz,ca2,cgly,chol,cl,cobalt2,cu2,cytd,fe3,fol,frmd,glcur,gln__L,gly_asn__L,glyc3p,ile__L,indole,k,lys__L,mg2,mn2,nmn,phe__L,progly,ribflv,so4,tol,tyr__L,uaccg,udcpp,val__L,zn2
all complete mro M11_Microcystis LalaDgluMdap,acmana,alaala,amp,arg__L,bz,ca2,cl,cobalt2,cu2,cytd,fe2,fe3,fol,frmd,gln__L,gly_asn__L,hom__L,ile__L,indole,k,lys__L,mg2,mn2,nmn,o2,phe__L,pnto__R,progly,ribflv,so4,tyr__L,uaccg,udcpp,val__L,zn2
all complete mro M11_Roseomonas arg__L,ca2,cgly,cl,cobalt2,cu2,cytd,fe3,glcur,glyb,glyc3p,k,lys__L,mg2,mn2,nmn,phe__L,so4,thm,tol,zn2

smetana -g M11_Microcystis.xml M11_Roseomonas.xml --molweight
/home/nico/miniconda3/lib/python3.8/site-packages/smetana/smetana.py:351: UserWarning: MRO: Failed to find a valid solution for: M11_Roseomonas
warn('MRO: Failed to find a valid solution for: ' + org_id)

Thanks for your help!

Nico

@cdanielmachado
Copy link
Owner

Hi Nico,

Your problems are related to these previously identified issues:

#17

#10

To be honest, there is not much that can be done without trying to make the formulation a bit more robust, which I won't have time to do in the near future.

One thing that might help (and I would always recommend) is to ignore inorganic compounds:

--exclude inorganic.txt

You can find the file here: https://www.dropbox.com/s/bapo01qf1uef3wm/inorganic.txt?dl=0

@ntromas
Copy link
Author

ntromas commented Apr 28, 2021 via email

@cdanielmachado
Copy link
Owner

Hi Nico,

Yes, I would recommend always using --molweight. From my experience, minimizing the total mass of the consumed substrates, rather than simply the total number of substrates, results in more realistic estimates of the growth medium.

Regarding the variation in the outputs, my suggestion is to run simulations a few times (10 or 100 depending on how long they take), and then just compute the average of the scores. And you can also use the variability of the score as an indication of how reliable it is.

@ntromas
Copy link
Author

ntromas commented Apr 30, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants