-
Notifications
You must be signed in to change notification settings - Fork 160
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
netCDF exporter does not contain any time steps #200
Comments
|
Transition between pysteps 1.3 and 1.4 introduced a bug in the callback function used in steps and sseps. The arrays supplied to the callback had incorrect shapes. The problem should be fixed with commit cf34300. |
Thanks a lot, @pulkkins! I'll test it and close the issue if everything works. |
Works perfectly! I'm closing this issue now. |
@pulkkins, @RubenImhoff thanks a lot for solving this issue. I'm re-opening it because I'd like to include a specific test for the netcdf exporter, so to make sure that we'd detect the same problem in case. |
Good idea, Daniele! |
I may have a potential issue:
I have a test script for pysteps to make ensemble nowcasts, just to check whether everything works fine. It saves the nowcasts in a netcdf, but the netcdf does not contain any timesteps in the rainfall data when using pysteps 1.4. It seems that this is caused by the changes in ./nowcasts/steps.py. If I use steps.py from version 1.3, the script works fine and the output netcdf contains two timesteps with the ensemble nowcast in it. I can't figure out why. Perhaps my script is just outdated, but maybe it is an actual bug. I have put the script in the comment below, it uses the KNMI data from the pysteps data repository, so it should be relatively easy to run.
The text was updated successfully, but these errors were encountered: