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

refactor: RNA formation, issue #354 #364

Merged
merged 10 commits into from
Feb 24, 2022

Conversation

PkiwiBird
Copy link
Contributor

Main improvements in this PR:

implement the suggestions of issue #354

  • add compartmentalised metabolite RNA[n] (MAM02847n)
  • change reaction MAR07161 (RNA creation) from cytoplasm to nucleus
  • add transport reaction of RNA from the nucleus to the cytoplasm (MAR13086)
  • move gene rule of reaction MAR08639 to reaction MAR13086, as it represents the nuclear pore complex
  • swap the direction of reaction MAR07162, as explained in issue RNA formation #354
  • remove the reaction of DNA transport between the nucleus and cytoplasm MAR08639, since DNA shouldn't go out of the nucleus and the reaction's gene rule was already associated to another reaction, MAR13086

Comments:

  • not sure what is the appropriate reaction score for reaction MAR13086
  • and could not find ids form other databases for this reaction, probably beacause RNA has to be first processed into mature RNA (with polyA-tail and 5'Cap) before moving to cytoplasm...

I hereby confirm that I have:

  • Tested my code on my own computer for running the model
  • Selected develop as a target branch

add RNA in nucleus
add reaction MAR13086, transport of RNA from nucleus to cytoplasm
implement the suggestions of issue SysBioChalmers#354
- add compartmentalised metabolite RNA[n] (MAM02847n)
- change reaction MAR07161 (RNA creation) from cytoplasm to nucleus
- add transport reaction of RNA from the nucleus to cytoplasm (MAR13086)
- move gene rule of reaction MAR08639 to reaction MAR13086, as it represents the nuclear pore complex
- swap the direction of reaction MAR07162, as explained in issue SysBioChalmers#354
- remove the reaction of DNA transport between the nucleus and cytoplasm MAR08639, since DNA shouldn't go out of the nucleus and the reaction's gene rule is no longer needed
delete reaction MAR08639 and add reaction MAR13086
Copy link
Collaborator

@JonathanRob JonathanRob left a comment

Choose a reason for hiding this comment

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

not sure what is the appropriate reaction score for reaction MAR13086

Eh, good question. I think you can use the same score as was used for MAR08639, which was a 2.

and could not find ids form other databases for this reaction, probably beacause RNA has to be first processed into mature RNA (with polyA-tail and 5'Cap) before moving to cytoplasm...

Yeah, it's a very simplified representation of the process, so if you were not able to find anything from other databases after searching for a bit, then I think it's OK to leave it as is.

model/reactions.tsv Show resolved Hide resolved
add deleted reaction MAR08639 to deprecatedReactions.tsv
score of reaction MAR13086 changed to 2
@mihai-sysbio mihai-sysbio changed the title Refactor: implement sugestions of issue # 354 refactor: RNA formation, issue #354 Feb 24, 2022
@mihai-sysbio mihai-sysbio merged commit 441bc5b into SysBioChalmers:develop Feb 24, 2022
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