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/issues/3786#issuecomment-590001709,https://api.github.com/repos/pydata/xarray/issues/3786,590001709,MDEyOklzc3VlQ29tbWVudDU5MDAwMTcwOQ==,327925,2020-02-22T21:50:46Z,2020-02-22T21:50:46Z,MEMBER,"If it's easier, a method for ""transpose these dimensions to the order which makes the data contiguous"" would meet my needs. I'm happy in principle to work on either feature, but when I've looked into contributing to xarray before, I've been a bit overwhelmed by complexity - I think I'm currently using a pretty small fragment of what it can do.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,568968607 https://github.com/pydata/xarray/issues/3786#issuecomment-589956473,https://api.github.com/repos/pydata/xarray/issues/3786,589956473,MDEyOklzc3VlQ29tbWVudDU4OTk1NjQ3Mw==,327925,2020-02-22T13:30:40Z,2020-02-22T13:30:40Z,MEMBER,"@max-sixty - I certainly want to avoid copying the data unless it's necessary. But I'd like to present the axes in the 'real' memory order, from the largest stride to the smallest. I appreciate that it shouldn't matter for program logic, but it can definitely matter for performance, and I know some users are going to use axes by position rather than by name, so I do consider it an important part of the API.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,568968607 https://github.com/pydata/xarray/issues/3786#issuecomment-589707231,https://api.github.com/repos/pydata/xarray/issues/3786,589707231,MDEyOklzc3VlQ29tbWVudDU4OTcwNzIzMQ==,327925,2020-02-21T15:40:04Z,2020-02-21T15:40:04Z,MEMBER,"Thanks, it makes sense that dimension order conceptually doesn't matter so much once they're labelled. Though I'd say that as Xarray has public APIs for accessing dimensions by position, changing how something like `.unstack()` orders them probably is a breaking change. I'm providing an API for other people to access data as a DataArray, and I'd like to have the dimensions in the order that makes it C contiguous, as a hint about what kinds of operations will be efficient. I know some users also go 'what is this complicated thing?' and extract the numpy array from it, in which case the order is more important. So another option that would work for me would be a separate method that rearranges the dimensions to the order that makes the array C contiguous.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,568968607 https://github.com/pydata/xarray/pull/3285#issuecomment-547837799,https://api.github.com/repos/pydata/xarray/issues/3285,547837799,MDEyOklzc3VlQ29tbWVudDU0NzgzNzc5OQ==,327925,2019-10-30T10:37:23Z,2019-10-30T10:37:23Z,MEMBER,"OK, I've xfailed it.","{""total_count"": 1, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 1, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490316894 https://github.com/pydata/xarray/pull/3285#issuecomment-547503432,https://api.github.com/repos/pydata/xarray/issues/3285,547503432,MDEyOklzc3VlQ29tbWVudDU0NzUwMzQzMg==,327925,2019-10-29T16:13:52Z,2019-10-29T16:13:52Z,MEMBER,"OK, looks like the test failure now is real. Let me know if you want me to comment out the relevant line so the tests pass.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490316894 https://github.com/pydata/xarray/pull/3285#issuecomment-547458156,https://api.github.com/repos/pydata/xarray/issues/3285,547458156,MDEyOklzc3VlQ29tbWVudDU0NzQ1ODE1Ng==,327925,2019-10-29T14:48:52Z,2019-10-29T14:48:52Z,MEMBER,"Nope. I don't understand the error, though it looks like astropy has had something similar: https://github.com/astropy/astropy/issues/6424 Also black is now failing on a number of files not affected here.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490316894 https://github.com/pydata/xarray/pull/3285#issuecomment-547453601,https://api.github.com/repos/pydata/xarray/issues/3285,547453601,MDEyOklzc3VlQ29tbWVudDU0NzQ1MzYwMQ==,327925,2019-10-29T14:40:05Z,2019-10-29T14:40:05Z,MEMBER,"Merged master, crossing fingers that fixes it.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490316894 https://github.com/pydata/xarray/pull/3285#issuecomment-540987510,https://api.github.com/repos/pydata/xarray/issues/3285,540987510,MDEyOklzc3VlQ29tbWVudDU0MDk4NzUxMA==,327925,2019-10-11T09:20:16Z,2019-10-11T09:20:16Z,MEMBER,"As in my other PR, one suggested addition causes a test failure, and I've put that in the last commit.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490316894 https://github.com/pydata/xarray/pull/3283#issuecomment-540981087,https://api.github.com/repos/pydata/xarray/issues/3283,540981087,MDEyOklzc3VlQ29tbWVudDU0MDk4MTA4Nw==,327925,2019-10-11T09:01:35Z,2019-10-11T09:01:35Z,MEMBER,"Yes, sorry, it dropped off my radar. I've implemented all the changes suggested above. The datetime64 & timedelta64 cases don't roundtrip completely, so there are test failures. I've added these in a separate commit at the end, so it's easy to cut that back out if you prefer.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490228661 https://github.com/pydata/xarray/pull/1220#issuecomment-273824430,https://api.github.com/repos/pydata/xarray/issues/1220,273824430,MDEyOklzc3VlQ29tbWVudDI3MzgyNDQzMA==,327925,2017-01-19T16:28:47Z,2017-01-19T16:28:47Z,MEMBER,"I just made ipython/ipython#10186 - if you've got time, could you try with that branch and see if it works without the config?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,201884828 https://github.com/pydata/xarray/pull/1220#issuecomment-273801697,https://api.github.com/repos/pydata/xarray/issues/1220,273801697,MDEyOklzc3VlQ29tbWVudDI3MzgwMTY5Nw==,327925,2017-01-19T15:12:55Z,2017-01-19T15:12:55Z,MEMBER,I'm still a bit puzzled why this is necessary. I couldn't reproduce it when building the docs locally.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,201884828 https://github.com/pydata/xarray/pull/124#issuecomment-42859001,https://api.github.com/repos/pydata/xarray/issues/124,42859001,MDEyOklzc3VlQ29tbWVudDQyODU5MDAx,327925,2014-05-12T17:04:17Z,2014-05-12T17:04:17Z,MEMBER,"Thanks @shoyer :+1: ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,33280730 https://github.com/pydata/xarray/pull/113#issuecomment-42624886,https://api.github.com/repos/pydata/xarray/issues/113,42624886,MDEyOklzc3VlQ29tbWVudDQyNjI0ODg2,327925,2014-05-09T01:17:46Z,2014-05-09T01:17:46Z,MEMBER,"Thanks, I've applied all of your suggestions. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,32919692 https://github.com/pydata/xarray/pull/113#issuecomment-42456498,https://api.github.com/repos/pydata/xarray/issues/113,42456498,MDEyOklzc3VlQ29tbWVudDQyNDU2NDk4,327925,2014-05-07T17:23:25Z,2014-05-07T17:23:25Z,MEMBER,"OK, I've excluded that combination from Travis. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,32919692 https://github.com/pydata/xarray/pull/113#issuecomment-42360034,https://api.github.com/repos/pydata/xarray/issues/113,42360034,MDEyOklzc3VlQ29tbWVudDQyMzYwMDM0,327925,2014-05-06T21:16:48Z,2014-05-06T21:16:48Z,MEMBER,"OK, let's see how that goes. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,32919692 https://github.com/pydata/xarray/issues/53#issuecomment-42258925,https://api.github.com/repos/pydata/xarray/issues/53,42258925,MDEyOklzc3VlQ29tbWVudDQyMjU4OTI1,327925,2014-05-06T01:17:16Z,2014-05-06T01:17:16Z,MEMBER,"Outstanding issue: What to do with unicode data (the native str type on Python 3 is unicode). The SciPy netcdf module doesn't attempt to handle unicode, so I wrote some code to encode unicode to bytes before storing it in scipy. That means, however, that the roundtrip doesn't work, because loading the data again gives bytes, not str. The netCDF4 module appears to handle this by just decoding any string data it finds to unicode, without considering what it's intended to be. Options: - Decode all string data to unicode on load. This is consistent with what netCDF already does. - Store the fact that the data was encoded in an attribute in the file. If that attribute is present, decode it again on load. - Don't decode any data, and change all the scipy roundtrip tests to expect that data comes back as bytes. I'm not familiar enough with netCDF and how it's used to know what makes sense here. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,28940534 https://github.com/pydata/xarray/issues/53#issuecomment-42229648,https://api.github.com/repos/pydata/xarray/issues/53,42229648,MDEyOklzc3VlQ29tbWVudDQyMjI5NjQ4,327925,2014-05-05T19:43:42Z,2014-05-05T19:43:42Z,MEMBER,"Work in progress: https://github.com/takluyver/xray/tree/py3 ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,28940534