-
-
Notifications
You must be signed in to change notification settings - Fork 76
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
VectorField does not reproject angle #295
Comments
I'll consider this as a bug, this should definitely be handled. |
I've had a quick go at implementing a VectorField project operation that takes into account the angle in #296. Do you think you could give that a try? |
Great! I've checked it out, see my small changes. On a slightly more general note, I must admit I am not super happy with using magnitude and angle instead of u,v for the VectorField plot. First, oceanographers/meteorologists and probably many other fields often use compass direction as angle (i.e., 0 degrees is North, 90 degrees is East), easily leading to confusion about what the angle is in a geographic context, and second, General Circulation Models usually return east/north velocity. Instruments may return angle/magnitude, but again, there are different conventions (for meteorologists, wind direction is where it is coming from, for oceanographers, current direction is where the flow goes). |
Absolutely agree with this, I think we should open an issue in HoloViews that either adds a mode to VectorField to switch between mag/angle and u/v or add an entirely separate element that accepts u,v components. |
At least, it would be a good thing that vectorfield accept a keyword like |
Same data, two different projections (PlateCarree and NorthPolarStereo). Note how the arrows are the same, just their locations changes. The obvious fix is to reproject before calculating magnitude and angle but it might be nice to mention this in the documentation.
Cartopy on the other hand (further below) seems to do this (using
quiver
), so it might be easy to implement (then, however, using East/North velocities instead of magnitude/angle:The text was updated successfully, but these errors were encountered: