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

Fixed some bugs in DEMETER/mgPipe #2350

Merged
merged 6 commits into from
Nov 7, 2024

Conversation

almut-heinken
Copy link
Contributor

@almut-heinken almut-heinken commented Oct 8, 2024

Fixed some bugs

  • file reading in DEMETER that vcauses the file to read wrong with certain model names
  • exchange reaction formulated wrong in DEMETER
  • plot in mgPipe that crashes if all models have the same number of microbes
  • diet models saved when not producing biomass
  • fixed bug in creation of violin plots
  • enabled entry of custom table and subset of taxa to create in createPanModels

I hereby confirm that I have:

  • Tested my code on my own machine
  • Followed the guidelines in the Contributing Guide
  • Selected develop as a target branch (top left drop-down menu)

(Note: You may replace [ ] with [X] to check the box)

@@ -26,7 +26,8 @@
% create the pan-models on species level
panPath=[pwd filesep 'panSpeciesModels'];

createPanModels(modPath,panPath,'Species','AGORA',numWorkers);
builtTaxa = {'Acinetobacter junii','Bacteroides ovatus','Enterobacter cloacae','Megasphaera elsdenii','Ruminococcus flavefaciens'};
createPanModels(modPath,panPath,'Species','AGORA_infoFile.xlsx',numWorkers,builtTaxa)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @almut-heinken
Is it possible that we move these models to COBRA.models and load them using "getDistributedModel" to avoid downloading them? or are these models updating regularly and it is important to always download their latest version?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @farid-zare
I modified the code to rely on a subset of models. I also opened a pull request on COBRA.models to upload the neccessary models.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a million @almut-heinken. This is great.

@rmtfleming rmtfleming merged commit ebbe615 into opencobra:develop Nov 7, 2024
0 of 2 checks passed
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

Successfully merging this pull request may close these issues.

3 participants