Skip to content

Human v1.1.1 - #134

Merged
haowang-bioinfo merged 30 commits into
masterfrom
devel
Sep 25, 2019
Merged

Human v1.1.1#134
haowang-bioinfo merged 30 commits into
masterfrom
devel

Conversation

@haowang-bioinfo

Copy link
Copy Markdown
Contributor

Main improvements in this PR:

JonathanRob and others added 30 commits May 27, 2019 17:06
Function accounts for changes to metabolite related fields in addition to reaction-related fields
This new function works with the new "docModelChanges" function.
To avoid clashing with the new "writecell" function introduced in MATLAB R2019a
These plotting functions were added 5 months ago, but this repo does not seem an appropriate location for such functions.
the "today" command requires the Financial Toolbox to be installed, whereas "clock" is in the base Matlab installation.
also fix ChEBI ID formatting
Annotation of model rxns, mets, and genes
This update is implemented by running script `curateExchangeRxns_issue117.m`:
- a number of exchange reactions and involved metabolites are modified, as addressed in #117
- 8 new exchange rxns are added
- HMR_9736 is determined as redundant and removed
- 4 mets are renamed, 8 new mets are added
- All modifications are saved to `curateExchangeRxns_issue117` and `curateExchangeRxns_issue117` for tracking the changes

@JonathanRob JonathanRob left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ok, these changes all look good, though I guess there is a conflict with some of the model files and the updateGrRules function that need to be resolved first.

@haowang-bioinfo

Copy link
Copy Markdown
Contributor Author

@JonathanRob yes, but it is easy to solve updateGrRules function. The tricky part is merging mat file between branches.

@haowang-bioinfo
haowang-bioinfo merged commit f126e28 into master Sep 25, 2019
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.

2 participants