Skip to content
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

Wrong data for Brasilia-Brazil #626

Open
leaot opened this issue Sep 17, 2021 · 5 comments
Open

Wrong data for Brasilia-Brazil #626

leaot opened this issue Sep 17, 2021 · 5 comments
Labels
bug fixed Fixed, wating for the feedback geolocation

Comments

@leaot
Copy link

leaot commented Sep 17, 2021

https://wttr.in and https://wttr.in/brasilia
are consistently showing data above the maximum historically recorded temperature 36.7 C
https://v2.wttr.in/brasilia
https://v3.wttr.in/brasilia
are showing data for Mato Grosso, a nearby state. Brasilia is Brazil's capital, located in the Federal District (DF)
The location for https://v4.wttr.in/brasilia
[-10.3333332,-53.1999999]
Is in the middle of Mato Grosso state
Screenshot from 2021-09-17 08-58-04

@chubin
Copy link
Owner

chubin commented Oct 31, 2021

Agree. Something is wrong here. I need to investigate it.
Could you please check if this location https://wttr.in/Guara is resolved correctly?
Are the coordinates in the Location: line correct?

@leaot
Copy link
Author

leaot commented Nov 1, 2021

Yes, https://wttr.in/Guara checks out for Guara city, Sao Paulo state (SP), let me know if I can help in any other way

@chubin
Copy link
Owner

chubin commented Nov 2, 2021

I believe I've fixed it. Please test. Now though, we have a different problem (#400), but it existed already.

@chubin chubin added the fixed Fixed, wating for the feedback label Nov 7, 2021
@leaot
Copy link
Author

leaot commented Nov 7, 2021

https://wttr.in looks good for Brasilia, Federal District, Brazil here. 'v2' and 'v3' are still linking to Cuiaba in Mato Grosso State.

@chubin
Copy link
Owner

chubin commented Nov 7, 2021

Good catch! I will fix it for v2/v3 too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fixed Fixed, wating for the feedback geolocation
Projects
None yet
Development

No branches or pull requests

2 participants