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

Replace fake metabolism section on compound report card with actual values. #298

Closed
mnowotka opened this issue Oct 13, 2017 · 5 comments
Closed

Comments

@mnowotka
Copy link
Contributor

Metabolism section on the compound report card currently always shows a metabolism of aspirine. This has to be replaces with actual real content.

@nclopezo
Copy link
Member

The section is now real, here are some examples:
https://chembl-glados.herokuapp.com/compound_report_card/CHEMBL25/
https://chembl-glados.herokuapp.com/compound_report_card/CHEMBL1200879
https://chembl-glados.herokuapp.com/compound_report_card/CHEMBL88/
https://chembl-glados.herokuapp.com/compound_report_card/CHEMBL109/

The preferred name of the substrates and metabolites is missing (chembl/chembl_webservices_2#149), as well as the enzyme chembl id (https://github.com/chembl/GLaDOS-es/issues/16)
After these 2 properties are added, this issue will be ready.

@mnowotka
Copy link
Contributor Author

Thanks a lot David, I'm on it! I'll update this thicket when chembl/chembl_webservices_2#149 is ready.

@nclopezo
Copy link
Member

Now that chembl/chembl_webservices_2#150 is ready, we need to update the index. After this, this issue should be ready. I am writing this so I don't forget.

@nclopezo
Copy link
Member

This is ready:
https://chembl-glados.herokuapp.com/compound_report_card/CHEMBL25/#Metabolism
The fields that were missing are now available.

@mnowotka
Copy link
Contributor Author

Thanks a lot @nclopezo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants