-
Notifications
You must be signed in to change notification settings - Fork 46
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
yeast 8.6.1 #321
Merged
yeast 8.6.1 #321
Changes from all commits
Commits
Show all changes
73 commits
Select commit
Hold shift + click to select a range
fb1d630
feat-rxns: add a unique subsystem for each rxn
e661e51
Fix: update the subsystem information
hongzhonglu 80fbf30
fix: correct subsystem for ATP consumption
hongzhonglu 7f83202
feat: Complex curation and annotation
cheng-yu-zhang 8be4ef3
feat: Complex curation and annotation
cheng-yu-zhang 123eef9
chore: rerun after merge, with RAVEN functions
edkerk a8db8b1
feat-rxns: add a unique subsystem for each rxn
e15537f
Fix: update the subsystem information
hongzhonglu 28cdacb
fix: correct subsystem for ATP consumption
hongzhonglu e8e25f6
fix: increaseVersion check file change
edkerk 95cab7c
reason_of_changes
cheng-yu-zhang 7f68ee7
latest
cheng-yu-zhang 8765d24
Merge branch 'complex-annotation' of https://github.com/SysBioChalmer…
cheng-yu-zhang 977c796
Update DBnewRxnsGenes.tsv
cheng-yu-zhang 4a14d90
feat:add new GPR for transport rxns
cheng-yu-zhang 38e0e2a
feat: change GPR for transport reaction r_2101
cheng-yu-zhang 5de9d8c
feat: correct 18 existing annotations of transport reactions and add …
cheng-yu-zhang 2101832
refactor: adjust data and code for RAVEN
edkerk 6a0556e
reasons of changes
cheng-yu-zhang bbaefdb
feat: refactor adding of new grRules
edkerk 1014a41
fix: remove old GPR curations
edkerk 9db28d9
chore: output model
edkerk 6b873a0
Merge branch 'develop' into complex-annotation
edkerk ce75352
feat: refactor adding of new grRules
edkerk 4bcfa9b
feat: reorganize files complex annotation
edkerk a271a05
chore: output model
edkerk 759801a
fix: accept change in README.md
edkerk 75598c5
add a higher level explanation and update "complexAnnotation" and "co…
cheng-yu-zhang 4d80ae3
update "Rxn_unique_subsystem"
cheng-yu-zhang b55b7c0
Merge branch 'unique_subsystem_of_rxn' of https://github.com/SysBioCh…
cheng-yu-zhang aaecdd6
refactor: essentialGenes no longer COBRA dependent
edkerk 7e36266
fix: remove word document
edkerk c9ddfce
chore: write new model files
edkerk 182cead
fix: remove COBRA dependency
edkerk eb80b90
update “transRxnNewGPR”
cheng-yu-zhang f01c2d2
chore: remove .idea folder
edkerk ec131ef
chore: update model files
edkerk 5de624b
fix: essentialGenes correct output
edkerk 9980a70
add_19_new_GPR
cheng-yu-zhang fca543a
Merge branch 'complex-annotation' into add_new_GPR
edkerk 8a643e6
Merge branch 'transporter-annotation-for-transport-rxns' into add_new…
edkerk 204b568
feat: consolidated curation between releases
edkerk dbaa751
chore: generate model files
edkerk b1912a7
feat: more info in readme.md when release is made
edkerk dea4961
feat: template curation script
edkerk d144c89
resolve the mentioned questions
cheng-yu-zhang 4306061
resolve the mentioned problems
cheng-yu-zhang f5390f0
Merge pull request #306 from SysBioChalmers/transporter-annotation-fo…
edkerk 5fee12b
Merge remote-tracking branch 'origin/develop' into complex-annotation
edkerk 63e1de0
chore: regenerate models
edkerk 42a3fb0
Merge pull request #305 from SysBioChalmers/complex-annotation
edkerk 7d08c62
Merge remote-tracking branch 'origin/develop' into add_new_GPR
edkerk bfb9e5d
chore: regenerate model files
edkerk 9f0b0aa
fix: curation script
edkerk 38b0bee
doc: contributing guidelines
edkerk bbd46f6
resolve the mentioned problem
cheng-yu-zhang eab45b9
remove r_4566 and make r_4232 reversible
cheng-yu-zhang ce5a927
refactor: use consolidate curation script
edkerk 43d4d14
feat: remove duplicate reactions
edkerk 8d36a96
Update subsystem files; add the "addRxnSubsystem.m"
cheng-yu-zhang d63a9ce
chore: remove backup file
edkerk fa480e0
refactor: remove capitalization of subsystems
edkerk a3bea04
Merge pull request #313 from SysBioChalmers/add_new_GPR
edkerk 25d5335
Merge remote-tracking branch 'origin/develop' into unique_subsystem_o…
edkerk c690e4d
fix: remove GPRs of deleted reactions
edkerk dba987b
chore: regenerated models
edkerk 3b1a641
Update uniqueSubsystems.tsv and delete keggpathway.tsv
cheng-yu-zhang ae0fa27
chore: generate new model files
edkerk 7ed59d5
refactor: reorganize old curation data and scripts
edkerk 7a25007
fix: capitalization gluconeogenesis
edkerk 5f49f66
Merge pull request #307 from SysBioChalmers/unique_subsystem_of_rxn
edkerk fdd0f2f
Merge branch 'main' into develop
edkerk dcf1cae
refactor: move requirements/ to code/requirements/
edkerk File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,23 +1,24 @@ | ||
### Description of the issue: | ||
*Try to be as clear as possible: Is it something wrong/missing in the model? Is it a simulation issue? PLEASE DELETE THIS LINE.* | ||
<!-- Try to be as clear as possible: Is it something wrong/missing in the model? Is it a simulation issue --> | ||
|
||
|
||
#### Expected feature/value/output: | ||
*How the reaction/metabolite/gene/simulation result should look (cite literature if needed). PLEASE DELETE THIS LINE.* | ||
<!-- How the reaction/metabolite/gene/simulation result should look (cite literature if needed). --> | ||
|
||
|
||
#### Current feature/value/output: | ||
*How the reaction/metabolite/gene/simulation actually looks in the `main` branch. PLEASE DELETE THIS LINE.* | ||
<!-- How the reaction/metabolite/gene/simulation actually looks in the `main` branch. --> | ||
|
||
|
||
#### Reproducing these results: | ||
*Please attach any code used below (if it's python code replace the word "matlab" with "python". PLEASE DELETE THIS LINE.* | ||
<!-- Please attach any code used below (if it's python code replace the word "matlab" with "python". --> | ||
```matlab | ||
|
||
``` | ||
|
||
**I hereby confirm that I have:** | ||
<!-- Note: replace [ ] with [X] to check the box --> | ||
- [ ] Tested my code with [all requirements](https://github.com/SysBioChalmers/yeast-GEM#required-software---user) for running the model | ||
- [ ] Done this analysis in the `main` branch of the repository | ||
- [ ] Done this analysis in the `develop` branch of the repository | ||
- [ ] Checked that a similar issue does not exist already | ||
- [ ] If needed, asked first in the [Gitter chat room](https://gitter.im/SysBioChalmers/yeast-GEM) about the issue |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
% This Function is for adding new subsystem into model. | ||
% Input: model, keggPathways.tsv. | ||
% NOTE: Before run the codes below, the file should be manually editted. | ||
% COBRA required. | ||
% New subsystem should be in .tsv format. | ||
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% | ||
|
||
% Load model | ||
cd .. | ||
model = loadYeastModel; | ||
|
||
%newsubsystem: | ||
fid = fopen('../data/modelCuration/v8_6_0/keggPathways.tsv'); | ||
sub_data = textscan(fid,'%s %s %s','Delimiter','\t','HeaderLines',1); | ||
id = sub_data{1}; | ||
subSystems = sub_data{2}; | ||
fclose(fid); | ||
for i = 1:length(model.rxns) | ||
if i <= length(id) | ||
subIndex = find(strcmp(model.rxns, id(i))); | ||
model.subSystems{subIndex, 1} = subSystems{i}; | ||
else | ||
model.subSystems{i, 1} = 'nan'; | ||
end | ||
end | ||
|
||
% Save model: | ||
cd .. | ||
saveYeastModel(model) | ||
cd modelCuration |
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,3 @@ | ||
## Missing Fields | ||
## Missing fields | ||
|
||
Scripts for adding fields to the model. | ||
|
||
**NOTE:** These scripts are provided as documentation + eventual re-use for developers. They are NOT intended to be used by regular users. | ||
Various scripts for adding and curating various annotation fields to the model, that are useful for model curation. These functions should be written in a generic format, so that they could be reused for future curations. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,9 @@ | ||
## Model Curation | ||
## Model curation | ||
|
||
Scripts for curating different parts of the model. | ||
|
||
**NOTE:** These scripts are provided as documentation + eventual re-use for developers. They are NOT intended to be used by regular users. | ||
Curations between each model release are gathered in consolidated curation scripts that are named after the model version that they are curation. E.g. v8_6_0.m includes updates that should be applied to model version 8.6.0 to result in the next model release. | ||
|
||
TEMPLATEcuration.m is the template script that should be used to generate a new consolidated curation script after a new model has just been released. | ||
|
||
In addition, this folder contains generic functions that are useful for model curation. These functions should be written in a generic format, so that they could be reused for future curations. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,64 @@ | ||
%% TEMPLATE FOR SCRIPT THAT CONSOLIDATES CURATIONS THAT ARE MADE TO A MODEL | ||
% RELEASE. EACH RELEASE (e.g. yeast-GEM v8.6.1) WILL HAVE ITS OWN SCRIPT WHERE | ||
% CURATIONS TO THAT MODEL RELEASE ARE CONSOLIDATED, MATCHING A DEDICATED FOLDER | ||
% WITH DATA IN code/modelCuration/. SEE EARLIER SCRIPTS AS EXAMPLE. AFTER A NEW | ||
% RELEASE IS MADE ON GITHUB, COPY-PASTE THIS TEMPLATE, REMOVE THIS LEADING | ||
% CAPITALIZED TEXT, AND REPLACE $VERSION WITH THE VERSION NUMBER OF THE RECENT | ||
% YEAST-GEM RELEASE (e.g. $VERSION --> 8.6.1). | ||
|
||
% This scripts applies curations to be applied on yeast-GEM release $VERSION. | ||
% Indicate which Issue/PR are addressed. If multiple curations are performed | ||
% before a new release is made, just add the required code to this script. If | ||
% more extensive coding is required, you can write a separate (generic) function | ||
% that can be kept in the /code/modelCuration folder. Otherwise, try to use | ||
% existing functions whenever possible. In particular /code/curateMetsRxnsGenes | ||
% can do many types of curation. | ||
|
||
%% Load yeast-GEM $VERSION (requires local yeast-GEM git repository) | ||
cd .. | ||
model = getEarlierModelVersion('$VERSION'); | ||
model.id='yeastGEM_develop'; | ||
cd modelCuration | ||
|
||
%% Brief description of curation to be performed (PR #xxx) [include correct PR or Issue number] | ||
% Potential longer description of curation to be performed. | ||
% If any data files need to be loaded, keep these in the dedicated folder at | ||
% data/modelCuration/v$VERSION | ||
|
||
% Example [DELETE WHEN NOT USED]: | ||
|
||
%% Curate gene association for transport rxns (PR #306) | ||
% Add new genes | ||
model = curateMetsRxnsGenes(model,'none','../../data/modelCuration/curationsOnV8_6_0/transRxnNewGPRGenes.tsv'); | ||
|
||
% Change GPR relations | ||
fid = fopen('../../data/modelCuration/curationsOnV8_6_0/TransRxnNewGPR.tsv'); | ||
changegpr = textscan(fid,'%q %q %q %q %q %q %q %q','Delimiter','\t','HeaderLines',1); | ||
newGPR.ID = changegpr{1}; | ||
newGPR.GPR = changegpr{2}; | ||
fclose(fid); | ||
|
||
model=changeGrRules(model,newGPR.ID,newGPR.GPR); | ||
|
||
% Delete unused genes (if any) | ||
model = deleteUnusedGenes(model); | ||
|
||
%% DO NOT CHANGE OR REMOVE THE CODE BELOW THIS LINE. | ||
% Show some metrics: | ||
cd ../modelTests | ||
disp('Run gene essentiality analysis') | ||
[new.accuracy,new.tp,new.tn,new.fn,new.fp] = essentialGenes(model); | ||
fprintf('Genes in model: %d\n',numel(model.genes)); | ||
fprintf('Gene essentiality accuracy: %.4f\n', new.accuracy); | ||
fprintf('Gene essentiality TP: %d\n', numel(new.tp)); | ||
fprintf('Gene essentiality TN: %d\n', numel(new.tn)); | ||
fprintf('Gene essentiality FP: %d\n', numel(new.fp)); | ||
fprintf('Gene essentiality FN: %d\n', numel(new.fn)); | ||
fprintf('\nRun growth analysis\n') | ||
R2=growth(model); | ||
fprintf('R2 of growth prediction: %.4f\n', R2); | ||
|
||
% Save model: | ||
cd .. | ||
saveYeastModel(model) | ||
cd modelCuration |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just wondering, is it expected to have
develop
here?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, new readme.md will be generated as part of making new release.