-
-
Notifications
You must be signed in to change notification settings - Fork 56
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
* bug in forecast.run_forecast: 'nwp_source' was not respected when loading the nwp data * adjustment of the test for run_forecast. Test for ICON and GFS NWP. * change data.get_nwp() in a way that the nwp data is available for dates more than 90 days in the past. The hisorical api from open meteo is used for this purpose. As this does not contain any visibility data, the value is set to the maximum of 24000 km. In addition, the 'openmeteo-requests' pip package is used. this is clearer and uses a binary data format instead of json so that the transmission is faster. And so the string tinkering for the api request can be omitted. * remove the request for user input. it is not necessary and makes testing the function unnecessarily complicated. * fix visibility bug and return to a 7 day nwp request just for better comparison * vis as float * cast hole df to float32 since the manual vis add with 24000 was casted as float64 * cast after setting the time index and cast to float64 instead float64 since the old modell also hold dtype float64 --------- Co-authored-by: Jakob Elias Gebler <[email protected]>
- Loading branch information
Showing
2 changed files
with
89 additions
and
60 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters