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

feat: memote #239

Merged
merged 255 commits into from
Apr 23, 2021
Merged

feat: memote #239

merged 255 commits into from
Apr 23, 2021

Conversation

mihai-sysbio
Copy link
Member

Main improvements in this PR:

As a follow-up to #162, this PR merges in the devel and feat/yaml-validation branches in order to convert the Travis CI pipeline that runs memote to GH Actions. Just as before, the history_report.html gets commited to the gh-pages branch automatically.

feiranl and others added 30 commits November 16, 2018 22:59
* Add new GPRs for transport rxns
* Add a function for updating those GPRs to model
* Update new GPRs for those transport rxns
Update the confidence scores for those reactions of transport rxns. Note: r_0475, r_1667, r_1729, r_1805, r_2025, r_2064, r_4403 ,r_4435, r_4484, r_4598, r_4599 rxn confidence scores were also changed due to they have gRrules
change the function to let Ca2+ can enter into the cell freely
update in the model
… update

fix the rxnconfidence scores for those reactions in this update:exchange reactions: NAN; reactions for modeling: 1
Fix biomass: Add Ca(2+) into Biomass
* remove rows with no formulas (redundant)
* remove undesired ' characters
* some bugs in function
as complex carbohydrates have varying length, it's best to use the monomeric form
fix-met.prop: correct metFormulas
This is to update the model to include fatty acid esters produced during alcoholic fermentation in the cytoplasm and extracellular membrane via the protein medium-chain fatty acid ethyl ester synthase/esterase 2.
@Midnighter
Copy link

I was thinking that I should add a GH workflow to memote in addition to or instead of the Travis CI one. I'm glad that you're already going ahead with that.

@mihai-sysbio mihai-sysbio changed the title Feat/memote feat: memote Apr 11, 2021
@mihai-sysbio
Copy link
Member Author

@Midnighter that would be great.

This very large PR has been in progress for a long time, but finally it passes all checks. It would be nice to merge it and let subsequent ideas/issues surface as different PRs.
Note: the fix/requirements PR #256 should be merged before this one.

@mihai-sysbio mihai-sysbio merged commit 2099e8c into SysBioChalmers:feat/memote Apr 23, 2021
@mihai-sysbio mihai-sysbio deleted the feat/memote branch April 27, 2021 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
memote anything related to the memote report
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants