search

KarrLab / datanator_rest_api

A OAS3 compliant REST API for the Datanator integrated database
MIT License
0 stars 3 forks source link

Kinetics endpoint is failing on the API, but not on the test API #111

Closed jonrkarr closed 4 years ago

jonrkarr commented 4 years ago

Working: https://testapi.datanator.info/reactions/kinlaw_by_rxn/?_from=0&size=1000&bound=loose&dof=0&substrates=YSYKRGRSMLTJNL-KFQCIAAJSA-N&products=PSXWNITXWWECNY-UCBTUHGZSA-N,XLYOFNOQVPJJNP-UHFFFAOYSA-N&taxon_distance=true&species=Escherichia%20coli

Not working: https://api.datanator.info/reactions/kinlaw_by_rxn/?_from=0&size=1000&bound=loose&dof=0&substrates=YSYKRGRSMLTJNL-KFQCIAAJSA-N&products=PSXWNITXWWECNY-UCBTUHGZSA-N,XLYOFNOQVPJJNP-UHFFFAOYSA-N&taxon_distance=true&species=Escherichia%20coli

jonrkarr commented 4 years ago

These failures seem to be fairly common. If this related to Heroku, then I think we should deal with #107 relatively soon.

lzy7071 commented 4 years ago

This was not an issue of Heroku. I haven't changed the master branch (datanator_query_python and datanator_rest_api) in a while because I was thinking once we finalized the first test version (test-frontend, test-rest-api, test-datanator-query-python), I would just merge the test branch into master branch.

jonrkarr commented 4 years ago

Got it. In that case, I'll start pushing to the dev branch rather than master. Then CircleCI will use the test API.

lzy7071 commented 4 years ago

These failures seem to be fairly common. If this related to Heroku, then I think we should deal with #107 relatively soon.

It was a result of changing some of the query functions from find in MongoDB to aggregate in MongoDB. The two functions return two different types of objects that need to be handled differently.