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

Difficult to read metabolic models when all small molecules are displayed #43

Open
ValWood opened this issue Jun 19, 2024 · 8 comments
Open
Labels
enhancement New feature or request

Comments

@ValWood
Copy link

ValWood commented Jun 19, 2024

The metabolic pathways are very hard to consume when they display all of the small molecules.
It would be better if there was a toggle to "collapse small molecules" so that you could see only the entities.

e.g
http://noctua.geneontology.org/workbench/noctua-alliance-pathway-preview/?model_id=gomodel%3AYeastPathways_TCA-EUK-PWY

http://noctua.geneontology.org/workbench/noctua-alliance-pathway-preview/?model_id=gomodel%3A633b013300000306

@vanaukenk vanaukenk added the enhancement New feature or request label Jun 25, 2024
@vanaukenk
Copy link

Also, we should filter out duplicates in the list on the right side of the model.

@vanaukenk
Copy link

vanaukenk commented Nov 20, 2024

I think display of metabolic models needs some more discussion to address:

  1. Possible different view with or without the chemicals
  2. If we don't display the chemicals, figure out a rule for displaying 'provides input for' in the gene-only display.
  3. Removing duplicate small molecules from the right-hand list.
  4. How to get these models to display on the Alliance pages, i.e. meet the minimal criteria for a 'causal' model.

@cmungall

@ValWood
Copy link
Author

ValWood commented Nov 20, 2024

If we don't display the chemicals, what causal relations would/should exist between the genes?

provides input for (rather than has input/has output). In the instances where we do add the chemical we use both...

@vanaukenk
Copy link

provides input for (rather than has input/has output). In the instances where we do add the chemical we use both...

Yes, sorry, that was more of a rhetorical question - this is our only relation option, but we'd need to figure out the display when that relation is not explicit in the model.

@ValWood
Copy link
Author

ValWood commented Nov 20, 2024

But you always need one or the other? Sometimes we add both (i.e in cases where we really want to include the chemical before they can be imputed)

Screenshot 2024-11-20 at 21 22 08

@ValWood
Copy link
Author

ValWood commented Nov 20, 2024

And the display works:

Screenshot 2024-11-20 at 21 25 06

@ValWood
Copy link
Author

ValWood commented Nov 20, 2024

Ok I get it now. If only the chemical connections are present you would need to infer the gene-gene relation...

@vanaukenk
Copy link

Ok I get it now. If only the chemical connections are present you would need to infer the gene-gene relation...

Exactly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants