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

Fix biomass: Add Ca(2+) into Biomass #183

Merged
merged 4 commits into from
Dec 6, 2018
Merged

Fix biomass: Add Ca(2+) into Biomass #183

merged 4 commits into from
Dec 6, 2018

Conversation

feiranl
Copy link
Collaborator

@feiranl feiranl commented Dec 4, 2018

Main improvements in this PR:

  • Continuing the PR New biomass #174
  • Add Ca(2+) into the biomass
  • update the function minimal_Y6.m to let Ca(2+) enter into the cell freely

I hereby confirm that I have:

  • Tested my code with all requirements for running the model
  • Selected devel as a target branch (top left drop-down menu)
  • If needed, asked first in the Gitter chat room about this PR

change the function to let Ca2+ can enter into the cell freely
update in the model
@feiranl feiranl requested a review from BenjaSanchez December 4, 2018 20:06
@BenjaSanchez BenjaSanchez added curation bug something is wrong in the model labels Dec 5, 2018
… update

fix the rxnconfidence scores for those reactions in this update:exchange reactions: NAN; reactions for modeling: 1
@BenjaSanchez BenjaSanchez merged commit 8aef65c into devel Dec 6, 2018
@BenjaSanchez BenjaSanchez mentioned this pull request Dec 6, 2018
@BenjaSanchez BenjaSanchez deleted the fix-biomass branch December 17, 2018 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something is wrong in the model
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants