chembl / GLaDOS

Web Interface for ChEMBL @ EMBL-EBI
https://www.ebi.ac.uk/chembl/
Other
46 stars 6 forks source link

Get protein target hierarchy from dedicated endpoint #1217

Closed nclopezo closed 4 years ago

nclopezo commented 4 years ago

Fixes https://github.com/chembl/GLaDOS/issues/1055 The trees are now calculated by the backend, and not built via aggregations directly on elasticsearch. Also, it is now possible to explore the 3 target hierarchies (protein targets, organism taxonomy, and gene ontology)

pep8speaks commented 4 years ago

Hello @nclopezo! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2019-12-04 11:24:05 UTC
chembl-bot commented 4 years ago

I am going to a start test server for 4b1f7ec8bcd3f22ad196d7f6f2f28e86ecc18017

chembl-bot commented 4 years ago

Starting Ghost inspector tests on http://wwwdev.ebi.ac.uk/chembl/test1/get-protein-target-hierarchy-from-dedicated-endpoint/

chembl-bot commented 4 years ago

Ghost inspector tests results, passed: False

Tested Commit: 4b1f7ec8bcd3f22ad196d7f6f2f28e86ecc18017

Suite Name: Basic

All passing: False

Suite Name: Report Cards

All passing: True

Suite Name: ReportCards - Compound

All passing: False

Suite Name: State URLs and Query editing

All passing: True

Suite Name: Downloads

All passing: False

Suite Name: Sequence and Structure Searches

All passing: False

Suite Name: Redirections

All passing: True

nclopezo commented 4 years ago

Tests were manually checked, Ghost inspector is hitting the test server too hard.