-
Notifications
You must be signed in to change notification settings - Fork 7
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
filter by journal doesn't appear to work as expected #89
Comments
@tracy55 Thank you for filing this. When querying the API, you cannot use the same GET parameters as you see in the frontend or the frontend query language. You need to use the Vespa YQL query language through the Vespa Search API. Some examples are listed in the CORD-19 documentation under "Example API queries". I'll update this issue with your exact query shortly. |
@tracy55 Your specific FE query would translate to something like this: Or - if using a POST request with the JSON (a bit easier to work with) it would look like this:
Hope this helps. |
Thank you @frodelu this is very helpful, much appreciated. |
@frodelu I'm noticing some latency, what is the expected sla for this sample call https://api.cord19.vespa.ai/search/?query=covid-19&summary=short&hits=20&ranking=freshness&yql=select+*+from+sources+*+where+userQuery%28%29+and+timestamp+%3E+1577836800%3B |
About 10-15 ms. Where are you calling us from, and do you use persistent connections? |
I would like to filter results by journal but the API does not seem to be respecting the filter and returns random journals while the FE search works as expected:
api: https://api.cord19.vespa.ai/search/?query=covid-19&summary=short&ranking=freshness&year=2020&fieldset=all&journal=The+Lancet
FE: https://cord19.vespa.ai/search?query=covid-19&summary=short&ranking=freshness&year=2020&fieldset=all&journal=The+Lancet
Am I doing something wrong?
The text was updated successfully, but these errors were encountered: