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

Allow long term hourly forecasts for supported models? #322

Open
Mrnofish opened this issue Jan 25, 2025 · 13 comments
Open

Allow long term hourly forecasts for supported models? #322

Mrnofish opened this issue Jan 25, 2025 · 13 comments

Comments

@Mrnofish
Copy link

A number of models can output hourly forecasts that greatly exceed the 72 hours limit in the app.

Ranging from 96 for Arpege to.the full 16 days e.g. some ECMWF models and GFS, with others falling in between such as JMA, UKMO, CMA Grapes, BOM Access and so on.

The setting in the preferences reads "number of hours shown in main view hourly chart” however the limit appears ro be applied to every hourly chart, including Compare Models and the dedicated Chart feature.

Is the maximum of 72 hours intended to maintain readability in the main view?

Would it be possible to extend the plot range for Chart and Compare Models to the open meteo maximum?

@farfromrefug
Copy link
Collaborator

@Mrnofish if I remember correctly (not in front of a computer) , open-meteo only goes up to 72 hours. If not I have no issue allowing more!

@Mrnofish
Copy link
Author

Mrnofish commented Jan 25, 2025

Perhaps it was changed along the way?

I did my homework, and was quite surprised myself but it is indeed the case.

The daily metrics look like they are actually based on the hourly forecasts, at least for some models. Maybe all models, and the hourlies are just not shared publicly.

@farfromrefug
Copy link
Collaborator

@Mrnofish will be added in next verison

@Mrnofish
Copy link
Author

Mrnofish commented Feb 7, 2025

I've upgraded to 160, however the setting only goes up to 168 (7 days) instead of 16?

Also the setting only applies to Compare Models and not Chart.

BTW setting the length of the hourly forecast down to the individual hour seems a bit overkill to me. I'd imagine it would be fine and more convenient to use 12 hours steps, or perhaps even 24, seeing how this is the increment Open Meteo Is using, too.

@Mrnofish
Copy link
Author

Mrnofish commented Feb 7, 2025

I just ran into an odd bug, after opening the app and refreshing manually I've got a:

"Error:

Forecast days is invalid. Allowed range 0 to 168.
Given 168.”

This in Main view, naturally.

@farfromrefug
Copy link
Collaborator

@Mrnofish i have seen some reports like that. i have made a fix

@0-5-0
Copy link
Contributor

0-5-0 commented Feb 18, 2025

Seems ok in the version released yesterday

@Mrnofish
Copy link
Author

@0-5-0 The 168 hours bug seems indeed gone but the questions mentioned in the previous comment, compare models over periods longer than a week and the unwieldy setting control remain unaddressed.

Since @farfromrefug is going to take a long break, I'm guessing this is going to remain the way it is for the foreseeable future and thus the ticket should stay open.

@farfromrefug
Copy link
Collaborator

@Mrnofish sorry i missed the part about compare models and unwieldy setting control.
can you detail a bit ? might have time to look at it this week.

@Mrnofish
Copy link
Author

I mean this part:

I've upgraded to 160, however the setting only goes up to 168 (7 days) instead of 16?

Also the setting only applies to Compare Models and not Chart.

BTW setting the length of the hourly forecast down to the individual hour seems a bit overkill to me. I'd imagine it would be fine and more convenient to use 12 hours steps, or perhaps even 24, seeing how this is the increment Open Meteo Is using, too.

@farfromrefug
Copy link
Collaborator

@Mrnofish not sure what you mean by this Also the setting only applies to Compare Models and not Chart. . The setting is applied to all Open-Meteo requests so it should be used everywhere

@Mrnofish
Copy link
Author

Right, looks like that particular observation is no longer accurate. The rest, however, still is.

The setting is applied to all Open-Meteo requests so it should be used everywhere

I don't think this is the case, the hourly chart in Main View shows at most 72 hours worth of data.

I'm not advocating for this to be changed, though: 72 hours could be more or less acceptable in that scenario. Allowing the full extent of the data to be visualized could be OK, too, as long as the app can figure a sane zoom level, so the default presentation of the graph is not too crowded to be legible.

@farfromrefug
Copy link
Collaborator

@Mrnofish ok i see what you mean! totally forgot about that setting! will change it!
BTW i wont change 12h steps for now. I know it is not that fluent to have to scroll through but at least everyone can choose whatever they want

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants