Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Quick fix for test_narr_example_variable_without_grid_mapping test (broken by xarray v0.14) #1206

Merged
merged 1 commit into from
Oct 16, 2019
Merged

Quick fix for test_narr_example_variable_without_grid_mapping test (broken by xarray v0.14) #1206

merged 1 commit into from
Oct 16, 2019

Commits on Oct 15, 2019

  1. Change whole dataset parse_cf from apply to merge 

    With xarray v0.14 (in particular, after xarray issue 3234), MetPy's usage of xarray.Dataset.apply results in a duplicated coordinate issue, like that observed in 2fe00b6. It seems like it is a very subtle index issue with xarray, however, it can be avoided entirely through using xarray.merge instead. The likely upstream issue remains, but this at least fixes MetPy for now.
    @jthielen
    jthielen committed Oct 15, 2019
    Configuration menu
    Copy the full SHA
    143c05b View commit details
    Browse the repository at this point in the history