Pratiman
Pratiman
@headtr1ck Thanks for the suggestions. I have added two tests (`ignore` and `warn`). Also, while testing, I found that a new argument broke `combine="nested"` due to invalid `ids`. I have...
Hi @headtr1ck, I have been thinking about the handling of `ids`. Current version looks like a patch work (I am not happy with it.). I think we can create ids...
@max-sixty Can you please go through the PR. Thanks!
It seems like one test failed `test_sparse_dask_dataset_repr (xarray.tests.test_sparse.TestSparseDataArrayAndDataset) `. It is not related to this PR.
@dcherian @max-sixty Can you please help me merge this request? I am not sure what more needs to be done. Also, every day I am getting this message. Not sure...
> can we fix the errors in the docs? I just rebased the branch. Let us see if this resolves the docs. Ahh.. found the problem.
@max-sixty all tests passed.
@kmuehlbauer Thank you for the suggestions and for fixing the typos. I have made changes based on your suggestions. Please check. For the tests, I did not write a test...
@kmuehlbauer I have thought about this approach before. However, it creates a problem with a 2x2 nested case. Therefore, I have settled for recreating the IDs and paths. You can...
@kmuehlbauer @max-sixty Any update on this PR? Thanks! -Pratiman