-
Notifications
You must be signed in to change notification settings - Fork 169
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
Optimizing the parameters for the optical flow calculation #376
Comments
This can be done using ray as suggested by @dnerini see example: https://colab.research.google.com/drive/1pMAJZ8AZe7gUDgCogenfdlc-ayX6kVVM?usp=sharing |
Lukas-Kanade from openCV provides some uncertainty information about the u,v components, can we use this? |
if possible, one by product of this issue could be adding this example as a tutorial to the pysteps documentation |
Hi!! As I mentioned yday ... I implemented in this example the INCA motion which uses the errors of previous nowcating to correct the latest kind-of-TREC motion vector computation. It uses numba but I am not sure this is required as a new library in pySTEPS but maybe we can think if these errors are useful for task 8 (velocity perturbation settings) or can help in optimizing the optical flow calculation. Here is the link to the example |
wouldn't this be rather a new feature to be implemented? |
I am not sure if you want me to implement these things in pySTEPS or not. I think the kind-of-TREC (using numba, so adding this dependency thing you mentioned) can be implemented in motion sublibrary. The previous nowcast errors computation and the introduction of those into the final vector motion can be also added but I am not sure the best place to do so. So any recommendation is welcome 😄 The think is that can be done and also understood as a correction to the error vector (so again in motion sublibrary) or as a preprocessing step (feature sublibrary). Also, in operations, this is not the way is done ... these errors are saved from the previous time steps. And last but not least, do we want to introduce the NWP wind correction as well? |
thanks! essentially the colab needs to be exported as jupyter notebook (.ipynb) and saved in the Then we can link it in the docs, same as the existing tutorial, see here: Line 35 in d77fe73
All this can be done in one PR, can you open one? I'll be happy to review it! |
Looks good. Maybe we can change Note to Caution? It's a bit more alerting, no? But this is a very minor comment |
[RMI/ KNMI / site specific] Is more better, and how to check this?
Number of frames, other parameters
The text was updated successfully, but these errors were encountered: