search

duckduckgo / zeroclickinfo-spice

DuckDuckGo Instant Answers based on JavaScript (JSON) APIs
https://duckduckhack.com/
Other
548 stars 942 forks source link

Weather: bad relevancy on the query "weather". #231

Closed nilnilnil closed 11 years ago

nilnilnil commented 11 years ago

I am in Paoli, PA.

screen shot 2013-10-05 at 3 17 27 pm

jagtalon commented 11 years ago

I got the right one for the snow plugin (New Jersey), but Minneapolis for the weather plugin. Is it a caching thing?

https://duckduckgo.com/js/spice/snow/Annandale%2C%20%20New%20Jersey%2C%20%20United%20States https://duckduckgo.com/js/spice/forecast/Minneapolis%2C%20United%20States

Now I got screen shot 2013-10-05 at 3 31 53 pm

yegg commented 11 years ago

I am also getting something completely random.

nilnilnil commented 11 years ago

I bet it needs a proxy_cache_valid directive.

yegg commented 11 years ago

https://github.com/duckduckgo/zeroclickinfo-spice/commit/b335477e7b632570b5ed40e4c30516c871021c2e

nilnilnil commented 11 years ago

Ah nice. Should have read more of my mail before responding! :smile:

yegg commented 11 years ago

Looks like the bug was internal and not in that line, but making sure before I close.