issue_comments: 954253465

This data as json

html_url issue_url id node_id user created_at updated_at author_association body reactions performed_via_github_app issue
https://github.com/pydata/xarray/pull/5886#issuecomment-954253465 https://api.github.com/repos/pydata/xarray/issues/5886 954253465 IC_kwDOAMm_X8444MCZ 35968931 2021-10-28T21:58:24Z 2021-10-28T21:58:24Z MEMBER

Did all of these changes print the warning?

I don't know - instead of testing all the cases I just made the .values->.to_numpy() change everywhere I could see. I think that makes sense given that: (a) there are tests elsewhere that should guarantee that those two methods are equivalent for non-pint arrays, (b) making those changes didn't break any existing tests, (c) .to_numpy() was intended to be the new default for calling within plotting code.

Would be nice if we could get these tested somehow.

I've added some tests for faceted line plots, faceted imshow, and faceted contourf. There is almost certainly some other type of plot or other edge case I haven't tested, but that's enough to show that this PR fixes what it intended to fix.

It would be nicer to know for certain that we are testing all the plot types with pint arrays, but I think that can maybe wait for another PR?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   1033884661