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/3545#issuecomment-1306152559,https://api.github.com/repos/pydata/xarray/issues/3545,1306152559,IC_kwDOAMm_X85N2k5v,775186,2022-11-07T20:33:02Z,2022-11-07T20:33:02Z,CONTRIBUTOR,I'm still along and yes I do still need this functionality (I still sync back to this PR when I have data missing vars). The issue was that the technical requirements got beyond what I was able to account for with the time I had available. If you or someone else was interested in picking it up I'd be happy to evaluate against my use cases. ,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,524043729
https://github.com/pydata/xarray/pull/3545#issuecomment-586629198,https://api.github.com/repos/pydata/xarray/issues/3545,586629198,MDEyOklzc3VlQ29tbWVudDU4NjYyOTE5OA==,775186,2020-02-15T18:38:25Z,2020-02-15T18:38:25Z,CONTRIBUTOR,"I just pushed an incomplete set of changes as @kmuehlbauer tests have demonstrated there was some incomplete cases the PR still isn't handling.
Here is a summary:
1. I've simplified the logic based on @dcherian comments but in order to keep the result deterministic needed to use list logic instead of set logic. I also kept the OrderedDict instead of going with the default dict as the built in ordering methods as of py 3.6 were still insufficient for keeping the ordering consistent (I needed to pop FIFO) which doesn't seem possible until py 3.8.
2. I did add a failing test to capture the cases @kmuehlbauer pointed out.
I'm not sure I have my head wrapped around xarray enough to address @dcherian's latest comments though which is why i'm sharing the code at this point. All tests are passing except the new cases which were pointed out.
I'll try to continue to get time to update this but wanted to at least provide this status update at this point as its been awhile.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,524043729
https://github.com/pydata/xarray/pull/3545#issuecomment-577773353,https://api.github.com/repos/pydata/xarray/issues/3545,577773353,MDEyOklzc3VlQ29tbWVudDU3Nzc3MzM1Mw==,775186,2020-01-23T17:00:06Z,2020-01-23T17:01:07Z,CONTRIBUTOR,@kmuehlbauer @dcherian @shoyer If it would be easier it could abandon this PR and resubmit a new one as the code has drastically changed since the original comments were provided? Essentially I'm waiting for feedback or approval of this PR.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,524043729
https://github.com/pydata/xarray/pull/3545#issuecomment-569842806,https://api.github.com/repos/pydata/xarray/issues/3545,569842806,MDEyOklzc3VlQ29tbWVudDU2OTg0MjgwNg==,775186,2019-12-31T01:18:35Z,2019-12-31T01:18:35Z,CONTRIBUTOR,"Hi, I've provided a new update to this PR (sorry it took me awhile both to get more familiar with the code and find the time to update the PR). I improved the logic to be a bit more performant and handle more edge cases as well as updated the test suite. I have a few questions:
1. The tests I wrote are a bit more verbose than the tests previously. I can tighten them down but I found it was easier for me to read the logic in this form. Please let me know what you prefer.
2. I'm still not quite sure I've captured all the scenarios as I'm a pretty basic xarray user so please let me know if there is still something I'm missing.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,524043729
https://github.com/pydata/xarray/pull/3545#issuecomment-555791968,https://api.github.com/repos/pydata/xarray/issues/3545,555791968,MDEyOklzc3VlQ29tbWVudDU1NTc5MTk2OA==,775186,2019-11-20T01:22:51Z,2019-11-20T01:22:51Z,CONTRIBUTOR,"Ok, I'll work on extending the updates with the feedback and additional tests. Thanks!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,524043729