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/1597#issuecomment-508802893,https://api.github.com/repos/pydata/xarray/issues/1597,508802893,MDEyOklzc3VlQ29tbWVudDUwODgwMjg5Mw==,1386642,2019-07-05T15:59:51Z,2019-07-05T15:59:51Z,CONTRIBUTOR,Thanks Joe!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-508802305,https://api.github.com/repos/pydata/xarray/issues/1597,508802305,MDEyOklzc3VlQ29tbWVudDUwODgwMjMwNQ==,2443309,2019-07-05T15:57:31Z,2019-07-05T15:57:31Z,MEMBER,Kudos to @nbren12 for sticking with this one! Looking forward to seeing this one in the wild. ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-508598650,https://api.github.com/repos/pydata/xarray/issues/1597,508598650,MDEyOklzc3VlQ29tbWVudDUwODU5ODY1MA==,1386642,2019-07-05T01:07:21Z,2019-07-05T01:07:21Z,CONTRIBUTOR,phew! Thanks for all the reviews and discussion everyone!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-508595343,https://api.github.com/repos/pydata/xarray/issues/1597,508595343,MDEyOklzc3VlQ29tbWVudDUwODU5NTM0Mw==,1217238,2019-07-05T00:33:07Z,2019-07-05T00:33:07Z,MEMBER,Thanks @nbren12!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-508593312,https://api.github.com/repos/pydata/xarray/issues/1597,508593312,MDEyOklzc3VlQ29tbWVudDUwODU5MzMxMg==,1386642,2019-07-05T00:09:43Z,2019-07-05T00:09:43Z,CONTRIBUTOR,It looks like the CI errors above aren’t related to this PR. There seems to be an issue with `to_pandas`.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-478700651,https://api.github.com/repos/pydata/xarray/issues/1597,478700651,MDEyOklzc3VlQ29tbWVudDQ3ODcwMDY1MQ==,24736507,2019-04-01T18:53:03Z,2019-07-04T20:50:21Z,NONE,"Hello @nbren12! Thanks for updating this PR. We checked the lines you've touched for [PEP 8](https://www.python.org/dev/peps/pep-0008) issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
##### Comment last updated at 2019-07-04 20:50:20 UTC","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-507510660,https://api.github.com/repos/pydata/xarray/issues/1597,507510660,MDEyOklzc3VlQ29tbWVudDUwNzUxMDY2MA==,22429695,2019-07-02T04:12:24Z,2019-07-04T20:50:06Z,NONE,"# [Codecov](https://codecov.io/gh/pydata/xarray/pull/1597?src=pr&el=h1) Report
> Merging [#1597](https://codecov.io/gh/pydata/xarray/pull/1597?src=pr&el=desc) into [master](https://codecov.io/gh/pydata/xarray/commit/e26fbca88cb6c3b950bd52879a193de85feef7c8?src=pr&el=desc) will **increase** coverage by `0.02%`.
> The diff coverage is `97.22%`.
```diff
@@ Coverage Diff @@
## master #1597 +/- ##
==========================================
+ Coverage 94.8% 94.83% +0.02%
==========================================
Files 67 67
Lines 12876 12912 +36
==========================================
+ Hits 12207 12245 +38
+ Misses 669 667 -2
```
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-507895012,https://api.github.com/repos/pydata/xarray/issues/1597,507895012,MDEyOklzc3VlQ29tbWVudDUwNzg5NTAxMg==,1217238,2019-07-03T00:45:55Z,2019-07-03T00:45:55Z,MEMBER,"OK, I plan to merge this shortly unless there are any objections :)","{""total_count"": 3, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 3, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-507822750,https://api.github.com/repos/pydata/xarray/issues/1597,507822750,MDEyOklzc3VlQ29tbWVudDUwNzgyMjc1MA==,1386642,2019-07-02T19:57:59Z,2019-07-02T19:57:59Z,CONTRIBUTOR,drat. Should be fixed now.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-507791978,https://api.github.com/repos/pydata/xarray/issues/1597,507791978,MDEyOklzc3VlQ29tbWVudDUwNzc5MTk3OA==,1217238,2019-07-02T18:23:17Z,2019-07-02T18:23:17Z,MEMBER,you have a lint error: https://github.com/pydata/xarray/pull/1597#issuecomment-478700651,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-507512420,https://api.github.com/repos/pydata/xarray/issues/1597,507512420,MDEyOklzc3VlQ29tbWVudDUwNzUxMjQyMA==,1386642,2019-07-02T04:23:10Z,2019-07-02T04:23:10Z,CONTRIBUTOR,"Okay. I responded to @benbovy's last comments and merged the upstream changes to master. How is this looking?
cc @jhamman ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-505533500,https://api.github.com/repos/pydata/xarray/issues/1597,505533500,MDEyOklzc3VlQ29tbWVudDUwNTUzMzUwMA==,1386642,2019-06-25T17:01:52Z,2019-06-25T17:01:52Z,CONTRIBUTOR,"> is there actual use cases where it is useful to provide multiple dimensions for sample_dims?
Yes! My main use-case does. For example, if you have a weather dataset where every lat/lon pair should be considered a separate ""sample"", you would use `sample_dims=['lat', 'lon']`.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-504623974,https://api.github.com/repos/pydata/xarray/issues/1597,504623974,MDEyOklzc3VlQ29tbWVudDUwNDYyMzk3NA==,1386642,2019-06-22T03:37:35Z,2019-06-22T03:37:35Z,CONTRIBUTOR,It looks like the tests passed. @benbovy How does it look now? Did I fix the issues you mentioned?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-504621993,https://api.github.com/repos/pydata/xarray/issues/1597,504621993,MDEyOklzc3VlQ29tbWVudDUwNDYyMTk5Mw==,1386642,2019-06-22T03:12:00Z,2019-06-22T03:12:00Z,CONTRIBUTOR,It looks like 43834ac8186a851b7 might have fixed this. Let's see if the tests pass.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-499915652,https://api.github.com/repos/pydata/xarray/issues/1597,499915652,MDEyOklzc3VlQ29tbWVudDQ5OTkxNTY1Mg==,1386642,2019-06-07T14:51:46Z,2019-06-07T14:54:25Z,CONTRIBUTOR,Does anybody have an idea why the `py36-dask-dev` tests are failing? None of the errors seems related to this PR.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-499789587,https://api.github.com/repos/pydata/xarray/issues/1597,499789587,MDEyOklzc3VlQ29tbWVudDQ5OTc4OTU4Nw==,1386642,2019-06-07T07:40:38Z,2019-06-07T07:40:38Z,CONTRIBUTOR,"@benbovy
Paradoxically I think making `to_stacked_array` accept the list of dimensions which won't be stacked is clearer. These *unstacked* dims needed to shared across all variables, which is a very simple requirement. I realized this after writing a bunch of error-prone validation code. See my last commit for more of my reasoning (5ca9a1d868a). I also updated the docs and tests accordingly.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-499761734,https://api.github.com/repos/pydata/xarray/issues/1597,499761734,MDEyOklzc3VlQ29tbWVudDQ5OTc2MTczNA==,1386642,2019-06-07T05:28:51Z,2019-06-07T05:28:51Z,CONTRIBUTOR,"@benbovy
> Is it expected behavior? I guess that not every combination is possible because we're not broadcasting here unlike to_array().
I believe so. In your third case, `to_stacked_array` basically wants to combine `a` and `b` into a DataArray with a dimension `y`. But that is not possible without broadcasting because `b` does not have a dimension `y`. I will try to add a more informative error message.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-494893796,https://api.github.com/repos/pydata/xarray/issues/1597,494893796,MDEyOklzc3VlQ29tbWVudDQ5NDg5Mzc5Ng==,1386642,2019-05-22T17:21:24Z,2019-05-22T17:21:24Z,CONTRIBUTOR,Thanks for your review @benbovy. I'll try to take a look soon.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-494470057,https://api.github.com/repos/pydata/xarray/issues/1597,494470057,MDEyOklzc3VlQ29tbWVudDQ5NDQ3MDA1Nw==,2448579,2019-05-21T16:47:30Z,2019-05-21T16:47:30Z,MEMBER,docs & tests LGTM. I don't understand stacking/unstacking/multiindexes too well...,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-494240726,https://api.github.com/repos/pydata/xarray/issues/1597,494240726,MDEyOklzc3VlQ29tbWVudDQ5NDI0MDcyNg==,2443309,2019-05-21T05:04:24Z,2019-05-21T05:04:24Z,MEMBER,@pydata/xarray - would love to get some more eyes on this PR. Its been lingering but @nbren12 seems to have addressed all prior comments.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-493249598,https://api.github.com/repos/pydata/xarray/issues/1597,493249598,MDEyOklzc3VlQ29tbWVudDQ5MzI0OTU5OA==,1386642,2019-05-16T22:11:55Z,2019-05-16T22:11:55Z,CONTRIBUTOR,Hey @rabernat. Did I resolve your comments?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-482645695,https://api.github.com/repos/pydata/xarray/issues/1597,482645695,MDEyOklzc3VlQ29tbWVudDQ4MjY0NTY5NQ==,2443309,2019-04-12T16:51:30Z,2019-04-12T16:51:30Z,MEMBER,@rabernat - you had asked for some changes in the early days of this PR. Are you satisfied with the current state here?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-482640504,https://api.github.com/repos/pydata/xarray/issues/1597,482640504,MDEyOklzc3VlQ29tbWVudDQ4MjY0MDUwNA==,1386642,2019-04-12T16:35:17Z,2019-04-12T16:35:17Z,CONTRIBUTOR,Hey. I just wanted to bump this PR. How does it look @rabernat @jhamman @shoyer?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-479128919,https://api.github.com/repos/pydata/xarray/issues/1597,479128919,MDEyOklzc3VlQ29tbWVudDQ3OTEyODkxOQ==,1386642,2019-04-02T18:15:10Z,2019-04-02T18:15:10Z,CONTRIBUTOR,Ok. I added an example which is very similar to the `DataArray.unstack` example.,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-478794367,https://api.github.com/repos/pydata/xarray/issues/1597,478794367,MDEyOklzc3VlQ29tbWVudDQ3ODc5NDM2Nw==,1386642,2019-04-02T00:20:24Z,2019-04-02T00:20:24Z,CONTRIBUTOR,"Okay. It looks like it is passing CI now, so I think it's ready for another look. How does it look? Are you still interested in including this functionality?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-478749910,https://api.github.com/repos/pydata/xarray/issues/1597,478749910,MDEyOklzc3VlQ29tbWVudDQ3ODc0OTkxMA==,1217238,2019-04-01T21:12:09Z,2019-04-01T21:12:09Z,MEMBER,"No worries, rebase is fine too :)
On Mon, Apr 1, 2019 at 2:09 PM Noah D Brenowitz
wrote:
> @shoyer Sorry. I rebased...
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> , or mute
> the thread
>
> .
>
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-478748969,https://api.github.com/repos/pydata/xarray/issues/1597,478748969,MDEyOklzc3VlQ29tbWVudDQ3ODc0ODk2OQ==,1386642,2019-04-01T21:09:31Z,2019-04-01T21:09:31Z,CONTRIBUTOR,@shoyer Sorry. I rebased...,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-478730862,https://api.github.com/repos/pydata/xarray/issues/1597,478730862,MDEyOklzc3VlQ29tbWVudDQ3ODczMDg2Mg==,1217238,2019-04-01T20:17:33Z,2019-04-01T20:17:33Z,MEMBER,"> @jhamman I did a little more work on this today. How do you recommend I update this to master? rebase?
We should add a section to our contributor guide on this, but the recommended workflow is just to use `git merge`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-478702680,https://api.github.com/repos/pydata/xarray/issues/1597,478702680,MDEyOklzc3VlQ29tbWVudDQ3ODcwMjY4MA==,1386642,2019-04-01T18:58:18Z,2019-04-01T18:58:18Z,CONTRIBUTOR,@jhamman I did a little more work on this today. How do you recommend I update this to master? rebase?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-445483774,https://api.github.com/repos/pydata/xarray/issues/1597,445483774,MDEyOklzc3VlQ29tbWVudDQ0NTQ4Mzc3NA==,1386642,2018-12-08T19:28:40Z,2018-12-08T19:28:40Z,CONTRIBUTOR,I'd be happy to pick this up again if you think it will go through.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-445291957,https://api.github.com/repos/pydata/xarray/issues/1597,445291957,MDEyOklzc3VlQ29tbWVudDQ0NTI5MTk1Nw==,2443309,2018-12-07T16:44:54Z,2018-12-07T16:44:54Z,MEMBER,@nbren12 - can we revive this? There or some conflicts but I think this was pretty close to done.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-366548976,https://api.github.com/repos/pydata/xarray/issues/1597,366548976,MDEyOklzc3VlQ29tbWVudDM2NjU0ODk3Ng==,1386642,2018-02-18T21:24:28Z,2018-02-18T21:24:28Z,CONTRIBUTOR,Sorry for random activity. I accidentally hard reset the master branch nbren12/xarray to pydata/xarray.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-366547666,https://api.github.com/repos/pydata/xarray/issues/1597,366547666,MDEyOklzc3VlQ29tbWVudDM2NjU0NzY2Ng==,16011037,2018-02-18T21:06:10Z,2018-02-18T21:06:10Z,CONTRIBUTOR,"Could not review pull request. It may be too large, or contain no reviewable changes.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-343718802,https://api.github.com/repos/pydata/xarray/issues/1597,343718802,MDEyOklzc3VlQ29tbWVudDM0MzcxODgwMg==,1217238,2017-11-12T07:29:22Z,2017-11-12T07:29:22Z,MEMBER,"@nbren12 OK, sounds good to me. Yes, it's a good idea to put more speculative stuff in other packages. I will take a look at your actual implementation.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-343643422,https://api.github.com/repos/pydata/xarray/issues/1597,343643422,MDEyOklzc3VlQ29tbWVudDM0MzY0MzQyMg==,1386642,2017-11-11T06:01:52Z,2017-11-11T06:02:12Z,CONTRIBUTOR,"@shoyer If you are okay with it, I think we might want to leave that to a later date if ever. I am not exactly sure what a useful API for that would be ATM. On the other hand, I have been using the original `stack_cat` and `unstack_cat` functions for a couple of months, and they have handled my basic uses pretty well.
For more complicated uses (e.g. taking different subsets of each variable and concatenating the output), I have started working on [a project](https://github.com/nbren12/xarray_sklearn) which is similar to [sklearn-pandas](https://github.com/scikit-learn-contrib/sklearn-pandas). Since there are a million ways several xarray variables could be processed/subsetted, stacked and then concatenated, I think this functionality should probably remain in a third party package for now.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-343626563,https://api.github.com/repos/pydata/xarray/issues/1597,343626563,MDEyOklzc3VlQ29tbWVudDM0MzYyNjU2Mw==,1217238,2017-11-11T01:04:03Z,2017-11-11T01:04:03Z,MEMBER,"@nbren12 in the linked issue (https://github.com/pydata/xarray/issues/1317#issuecomment-337796691) you wrote:
> After using my own version of this code for the past month or so, it has occurred to me that this API probably will not support stacking arrays of with different sizes along shared arrays. For instance, I need to ""stack"" humidity below an altitude of 10km with temperature between 0 and 16 km. IMO, the easiest way to do this would be to change these methods into top-level functions which can take any dict or iterable of datarrays. We could leave that for a later PR of course.
Is this something that would make sense to resolve before we merge this?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-343312698,https://api.github.com/repos/pydata/xarray/issues/1597,343312698,MDEyOklzc3VlQ29tbWVudDM0MzMxMjY5OA==,1386642,2017-11-09T22:28:22Z,2017-11-09T22:28:22Z,CONTRIBUTOR,Okay. I think I'm done with the updates to the documentation. ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-342665601,https://api.github.com/repos/pydata/xarray/issues/1597,342665601,MDEyOklzc3VlQ29tbWVudDM0MjY2NTYwMQ==,1386642,2017-11-08T00:04:57Z,2017-11-08T00:04:57Z,CONTRIBUTOR,"I just added some docs, but need to add to whats-new still,","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-338303735,https://api.github.com/repos/pydata/xarray/issues/1597,338303735,MDEyOklzc3VlQ29tbWVudDMzODMwMzczNQ==,2443309,2017-10-20T19:40:12Z,2017-10-20T19:40:47Z,MEMBER,@nbren12 - can you add these methods to `api.rst` and add a note to whats-new (in addition to adding some docs to the reshaping page)?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-333337845,https://api.github.com/repos/pydata/xarray/issues/1597,333337845,MDEyOklzc3VlQ29tbWVudDMzMzMzNzg0NQ==,1386642,2017-09-30T21:41:58Z,2017-09-30T21:41:58Z,CONTRIBUTOR,@rabernat Your point is well taken. I will add some docs/motivation to the reshaping page.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-333275551,https://api.github.com/repos/pydata/xarray/issues/1597,333275551,MDEyOklzc3VlQ29tbWVudDMzMzI3NTU1MQ==,1386642,2017-09-30T02:20:41Z,2017-09-30T02:21:14Z,CONTRIBUTOR,"Okay. I just changed the names of the methods and wrote a test case for the problem with the dtype of the stacked dimensions not being preserved by `to_stacked_array`.
At the moment, I am filling in the missing dimensions with None, so the resulting index has `dtype object'`. This is then concatenated with the bona fide indices of the variables which are not missing this index, so the index as a whole is cast to the lowest common denominator, which is `object`. Unfortunately we can't just put `Nan` in because, `NaN` is a floating point number thing, so I don't think all numpy dtypes have the equivalent of NaN. @shoyer Do you have any thoughts about how to resolve this?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-333271025,https://api.github.com/repos/pydata/xarray/issues/1597,333271025,MDEyOklzc3VlQ29tbWVudDMzMzI3MTAyNQ==,1386642,2017-09-30T01:06:09Z,2017-09-30T01:06:09Z,CONTRIBUTOR,"That naming sounds good to me.
Also, I was having an issue with `unstack_cat` returning an index with dtype `object` that I would like to sort out.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-333254137,https://api.github.com/repos/pydata/xarray/issues/1597,333254137,MDEyOklzc3VlQ29tbWVudDMzMzI1NDEzNw==,1217238,2017-09-29T22:31:03Z,2017-09-29T22:31:03Z,MEMBER,"If we have a separate method for `to_stacked_array()`, we should have a separate method for unstacking, too.
I like `DataArray.to_unstacked_dataset()` as the complement to `Dataset.to_stacked_array()`. I agree that it should be a DataArray method for chain-ability.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-333023152,https://api.github.com/repos/pydata/xarray/issues/1597,333023152,MDEyOklzc3VlQ29tbWVudDMzMzAyMzE1Mg==,1386642,2017-09-29T03:38:07Z,2017-09-29T03:38:07Z,CONTRIBUTOR,Or maybe `DataArray.to_dataset_unstacked` is good.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-333022698,https://api.github.com/repos/pydata/xarray/issues/1597,333022698,MDEyOklzc3VlQ29tbWVudDMzMzAyMjY5OA==,1386642,2017-09-29T03:33:34Z,2017-09-29T03:34:02Z,CONTRIBUTOR,"Thanks! Yah...I'm not very good at naming things. I think `Dataset.to_stacked_array` makes sense, but I would typically expect something like `from_stacked_array` to be a static method of `Dataset` (e.g. `pd.MultiIndex.from_tuples`). We could always move unstack_cat to `Dataset`, but then we miss out on the ability to make sequential method calls.
IMO the behavior of unstack_cat is kind of what I would expect `DataArray.to_dataset` to do in the case where `dim` is a stacked coordinate, so maybe we could put it in there.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958
https://github.com/pydata/xarray/pull/1597#issuecomment-332950910,https://api.github.com/repos/pydata/xarray/issues/1597,332950910,MDEyOklzc3VlQ29tbWVudDMzMjk1MDkxMA==,2443309,2017-09-28T20:13:50Z,2017-09-28T20:13:50Z,MEMBER,I want to let @shoyer and @rabernat weigh in on the API here. ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,261131958