search

materialsproject / api

New API client for the Materials Project
https://materialsproject.github.io/api/
Other
104 stars 38 forks source link

Issue with summary.search #805

Closed cpashartis closed 1 year ago

cpashartis commented 1 year ago

Hello again,

Is it possible there is some misconnection recently? Whenever I, or another use from my group tries to use the mprester.summary.search, we only get errors of :

MPRestError: REST query returned with error status code 404 on URL https://api.materialsproject.org/summary/?nelements_min=1&nelements_max=1&elements=Si&_all_fields=True&_limit=1000 with message: Response {"message":"no Route matched with those values"}`

where this was generated with: mpr.summary.search(elements=['Si'], num_elements=1)

version: 0.30.11

GuDazi-6666 commented 1 year ago

Hello, I have encountered the same problem as yours, Have you solved it？ @cpashartis @

cpashartis commented 1 year ago

I have not unfortunately, I have not investigated what the issue is exactly. All I know is this should ideally work...

GuDazi-6666 commented 1 year ago

today it works when I upgraded the package. But it mentioned that a package didn't match even it can work properly. I am not sure this is a correct solution.

munrojm commented 1 year ago

Sorry for the delayed response. Indeed, your issues can be fixed by updating the package to the latest release.

cpashartis commented 1 year ago

This seems to work now, thanks! Out of curiosity, would installing just this repo work as well in the future?

munrojm commented 1 year ago

It might, sometimes the repo will get a bit out of sync with the official release though. It is probably best to just install the latest release if you can. We always make sure it is compatible with the current version of the API.

janosh commented 1 year ago

Sounds like this is fixed in recent versions.