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

Open-Meteo as weather provider instead of OpenWeatherMap #297

Open
woheller69 opened this issue Feb 28, 2025 · 4 comments
Open

Open-Meteo as weather provider instead of OpenWeatherMap #297

woheller69 opened this issue Feb 28, 2025 · 4 comments
Labels
enhancement New feature or request good first issue Good for newcomers skill Skill request or bug

Comments

@woheller69
Copy link

Open-Meteo is open source and has excellent data, e.g. from Deutscher Wetterdienst.
https://github.com/open-meteo/open-meteo

@Stypox Stypox added enhancement New feature or request skill Skill request or bug good first issue Good for newcomers labels Feb 28, 2025
@woheller69
Copy link
Author

...and they are not only open-source but also do not track their users. Also they are responsive in case of issues or suggestions.
OpenWeatherMap is as "open" as OpenAI.

@papjul
Copy link
Contributor

papjul commented Mar 3, 2025

Open-Meteo misses the following points compared to OpenWeather:

  • Reverse geocoding (city name)
  • Icons from an URL

Not sure how you want to proceed for this.

I also wonder if someone wants to work on it, should they get rid of OpenWeather or make it configurable to choose?

@woheller69
Copy link
Author

what do you mean with "icons from an url"?

And does Dicio need reverse geocoding?

"How is the weather in Paris?" does not need it in my view.

I would suggest to make it the default option.

@papjul
Copy link
Contributor

papjul commented Mar 3, 2025

what do you mean with "icons from an url"?

And does Dicio need reverse geocoding?

https://github.com/Stypox/dicio-android/blob/v2.0/app/src/main/kotlin/org/stypox/dicio/skills/weather/WeatherSkill.kt#L51-L54

It's also missing the description, now that I look at it.
(Would require a mapping between a weather code and a translation)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers skill Skill request or bug
Projects
None yet
Development

No branches or pull requests

3 participants