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

Fix: Remove reaction MAR02517 #432

Merged
merged 3 commits into from
Nov 17, 2022
Merged

Fix: Remove reaction MAR02517 #432

merged 3 commits into from
Nov 17, 2022

Conversation

JonathanRob
Copy link
Collaborator

Main improvements in this PR:

This PR addresses Issue #381 by removing the reaction MAR02517. There is no evidence that the reaction takes place in the compartment where it was located (lysosome).

Four of the metabolites in MAR02517 (MAM02444l, MAM02678l, MAM00890l, and MAM01596l) were involved in no other reactions, and therefore became disconnected/unused after the reaction was removed. These four metabolites were therefore also removed from the model.

In addition, the EC code for reaction MAR02519 (an identical reaction to MAR02517 but in the endoplasmic reticulum compartment) was corrected to 2.3.1.199.

@JonathanRob JonathanRob changed the base branch from main to develop November 16, 2022 08:14
Copy link
Member

@haowang-bioinfo haowang-bioinfo left a comment

Choose a reason for hiding this comment

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

looks good

Copy link
Collaborator

@feiranl feiranl left a comment

Choose a reason for hiding this comment

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

Looks good to me too!

@haowang-bioinfo haowang-bioinfo merged commit 130e0e7 into develop Nov 17, 2022
@haowang-bioinfo haowang-bioinfo deleted the fix_issue381 branch November 17, 2022 07:22
@haowang-bioinfo haowang-bioinfo mentioned this pull request Dec 21, 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