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/782#issuecomment-195492662,https://api.github.com/repos/pydata/xarray/issues/782,195492662,MDEyOklzc3VlQ29tbWVudDE5NTQ5MjY2Mg==,1217238,2016-03-11T18:40:56Z,2016-03-11T18:40:56Z,MEMBER,"Works for me

On Fri, Mar 11, 2016 at 10:40 AM, Joe Hamman notifications@github.com
wrote:

> So do we want to remove _infer_interval_breaks all together in this PR?
> I'm leaning toward merging this as is and following up with another PR to
> handle the unevenly spaced coordinates in the future.
> 
> —
> Reply to this email directly or view it on GitHub
> https://github.com/pydata/xarray/pull/782#issuecomment-195492398.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,138086327
https://github.com/pydata/xarray/pull/782#issuecomment-195492398,https://api.github.com/repos/pydata/xarray/issues/782,195492398,MDEyOklzc3VlQ29tbWVudDE5NTQ5MjM5OA==,2443309,2016-03-11T18:40:03Z,2016-03-11T18:40:03Z,MEMBER,"So do we want to remove `_infer_interval_breaks` all together in this PR?  I'm leaning toward merging this as is and following up with another PR to handle the unevenly spaced coordinates in the future.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,138086327
https://github.com/pydata/xarray/pull/782#issuecomment-195455648,https://api.github.com/repos/pydata/xarray/issues/782,195455648,MDEyOklzc3VlQ29tbWVudDE5NTQ1NTY0OA==,1217238,2016-03-11T17:03:41Z,2016-03-11T17:03:41Z,MEMBER,"This PR looks good to me.

Regarding the larger issue of not inferring bounds, I guess I would be on board with that change, too. The original idea was to make pcolormesh and imshow consistent (other than the underlying plotting engine) and to always plot that last row/column of data, but maybe it's better to force users to be explicit. At the very least, it is clearly a bad idea to adjust bounds if they are not equally spaced (but I agree that guessing is bad default behavior).
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,138086327
https://github.com/pydata/xarray/pull/782#issuecomment-195066916,https://api.github.com/repos/pydata/xarray/issues/782,195066916,MDEyOklzc3VlQ29tbWVudDE5NTA2NjkxNg==,2443309,2016-03-10T21:51:08Z,2016-03-10T21:51:08Z,MEMBER,"any more comments here?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,138086327
https://github.com/pydata/xarray/pull/782#issuecomment-192037109,https://api.github.com/repos/pydata/xarray/issues/782,192037109,MDEyOklzc3VlQ29tbWVudDE5MjAzNzEwOQ==,2443309,2016-03-04T00:38:12Z,2016-03-04T00:38:12Z,MEMBER,"Short of adding cartopy to our continuous integration setup, does anyone have any ideas on how to test this change more robustly than I have shown here? I want to avoid testing the behavior of the either `matplotlib` or `Cartopy` and since in this case, we're choosing not to modify their behavior, I'm not sure we need to test further.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,138086327
https://github.com/pydata/xarray/pull/782#issuecomment-191840829,https://api.github.com/repos/pydata/xarray/issues/782,191840829,MDEyOklzc3VlQ29tbWVudDE5MTg0MDgyOQ==,950575,2016-03-03T16:32:20Z,2016-03-03T16:33:54Z,CONTRIBUTOR,"What if there are bounds in the file and the data is regularly spaced? I consider the current behavior a guess, and guessing should be an active user choice, not the automatic behavior. 
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,138086327
https://github.com/pydata/xarray/pull/782#issuecomment-191835686,https://api.github.com/repos/pydata/xarray/issues/782,191835686,MDEyOklzc3VlQ29tbWVudDE5MTgzNTY4Ng==,2443309,2016-03-03T16:21:38Z,2016-03-03T16:21:38Z,MEMBER,"> BTW I'd rather not have the _infer_interval_breaks at all.

Me too, but I don't mind doing it when we're not projecting the data.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,138086327
https://github.com/pydata/xarray/pull/782#issuecomment-191607244,https://api.github.com/repos/pydata/xarray/issues/782,191607244,MDEyOklzc3VlQ29tbWVudDE5MTYwNzI0NA==,950575,2016-03-03T06:35:02Z,2016-03-03T06:35:02Z,CONTRIBUTOR,"Bare in mind that some netCDF4 files do have the bounds data and those should be used as ""breaks"" when available.

BTW I'd rather not have the `_infer_interval_breaks` at all, not sure if assuming linear at the mid points are generic enough.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,138086327