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

[fmiweather] Add support for edited Scandinavia forecast #17555

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jlaur
Copy link
Contributor

@jlaur jlaur commented Oct 13, 2024

Resolves #17548

Resolves openhab#17548

Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
@jlaur jlaur added enhancement An enhancement or new feature for an existing add-on additional testing preferred The change works for the pull request author. A test from someone else is preferred though. labels Oct 13, 2024
@jlaur jlaur marked this pull request as ready for review October 15, 2024 18:13
@jlaur jlaur requested a review from ssalonen as a code owner October 15, 2024 18:13
Copy link
Contributor

@ssalonen ssalonen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you once again! Great improvement

small comment on readme

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please mention this in the intro paragraph, we could shortly introduce this better forecast model

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps you or @masipila can help me out a bit here? I only know that it's a different forecast edited by meteorologists, but I'm not into the details.

I can add that when testing with my own location, the two forecasts were exactly the same. When I changed to Helsinki, I could see differences between them.

@jlaur jlaur removed the additional testing preferred The change works for the pull request author. A test from someone else is preferred though. label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fmiweather] Add possibility to use fmi::forecast::edited in addition to fmi::forecast::harmonie
2 participants