-
Notifications
You must be signed in to change notification settings - Fork 11
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
Cartopy bug when plotting quivers #628
Comments
Added functions in plot_util.py: |
I've taken this a little further than the bug states. velocity_polar() fixes the cartopy bug but id doesn't make sense to just fix the bug without the other utilities for plotting NEMO velocities as quivers. |
I've added some unit tests for each function. I need to make sure they work. |
I've run the tests and they pass. I'm going to check how it works with some real NEMO data before suggesting to merge. |
I'm having problems running the build tests on Windows (Windows fatal exception: access violation). I've only added code not changed any so don't expect the tests to fail. Could someone else check the tests for me. |
I have been able to run most of the tests. test_gridded_initialisation and test_tidegauge_methods don't seem to run on my laptop, I don't think it is anything related to the functions I've added just weird windows things. |
…28_cartopy_quivers Bug/#628 cartopy quivers
There is a bug in cartopy when plotting quivers using a stereographic projection. This is hard to understand when a correction to the quivers is also needed for plotting NEMO currents over the North Pole. There is a correction that can be added to when plotting currents over the pole. The bug has been un-resolved in cartopy for several years.
There are additional functions needed to enable plotting NEMO velocities in general so that the curvilinear grid i, j currents are projected in northwards and westwards currents that can be plotted in CartoPy.
Branch made bug/#628_cartopy_quivers
AC:
The text was updated successfully, but these errors were encountered: