Closed PkiwiBird closed 2 years ago
Thank you for this, @PkiwiBird - super helpful.
I agree with all of your suggestions. Regarding your 2nd suggestion:
model could have an RNA[n] metabolite and create a transport reaction from the nucleus to the cytoplasm ...
we could consider copying the gene rule from MAR08639
(DNA transport between cytoplasm and nucleus, which should eventually be deleted based on your discussion in #350) as the gene rule for the new reaction to transport RNA from nucleus to cytoplasm.
Thank you for this, @PkiwiBird - super helpful.
I agree with all of your suggestions. Regarding your 2nd suggestion:
model could have an RNA[n] metabolite and create a transport reaction from the nucleus to the cytoplasm ...
we could consider copying the gene rule from
MAR08639
(DNA transport between cytoplasm and nucleus, which should eventually be deleted based on your discussion in #350) as the gene rule for the new reaction to transport RNA from nucleus to cytoplasm.
I'll make a pull request for those changes then. In that case, @JonathanRob do you think reaction MAR07162 should have the opposite direction only (Pi[c] + RNA[c] => 0.18 ADP [c] + 0.3 CDP [c] + 0.34 GDP [c] + 0.18 UDP [c])? Because that enzyme in the cytoplasm seems to only degrade RNA. It seems to have a polyadenylation role, but only shown up till now with mitochondrial mRNAs (here)
Sure, you can swap the direction on MAR07162
to only degrade the RNA, since you've provided evidence to support such a change.
Thank you @PkiwiBird for so very thoroughly describing the issue and for implementing the suggestions in the pull request, and thanks @JonathanRob for the review. And with the PR merged, I think this issue can be closed.
Description of the issue:
Expected feature/value/output:
Python code:
I hereby confirm that I have:
main
branch of the repository