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/2690#issuecomment-867461205,https://api.github.com/repos/pydata/xarray/issues/2690,867461205,MDEyOklzc3VlQ29tbWVudDg2NzQ2MTIwNQ==,14808389,2021-06-24T08:51:36Z,2021-06-24T08:51:36Z,MEMBER,"this has been unintentionally closed while renaming `main`, probably because the feature branch was removed. Feel free to reopen if you want to work on this in the future.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645
https://github.com/pydata/xarray/pull/2690#issuecomment-456210705,https://api.github.com/repos/pydata/xarray/issues/2690,456210705,MDEyOklzc3VlQ29tbWVudDQ1NjIxMDcwNQ==,5635139,2019-01-21T22:06:04Z,2019-01-22T15:59:08Z,MEMBER,"The only other features I could imagine adding would be a coord over multiple dimensions, and a multi-indexed dimension (but potentially we leave the latter out until we're completely confident about the conventions)
Agree on the names!
@TomNicholas if you want to split this up at all lmk
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645
https://github.com/pydata/xarray/pull/2690#issuecomment-456205288,https://api.github.com/repos/pydata/xarray/issues/2690,456205288,MDEyOklzc3VlQ29tbWVudDQ1NjIwNTI4OA==,1217238,2019-01-21T21:38:05Z,2019-01-21T21:38:05Z,MEMBER,"Maybe having a simple example available under just `xarray.testing.dummy_dataset()` is a good idea after all. If we do ever want to change it for xarray's own testing purposes, that would be quite easy.
That said... if we're going to emesh this in our API lets at least think a little bit to make sure we cover most of the special cases we care about. I think we cover most of the dtypes we care about, but let's also add at least:
- Some global attributes
- A scalar data variable (it's easy to forget about these)
Anything else?
We might also consider tweaking the names -- `dim1`, `dim2` and `dim3` is not very descriptive. Even `x`, `y` and `z` would be better.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645
https://github.com/pydata/xarray/pull/2690#issuecomment-456200066,https://api.github.com/repos/pydata/xarray/issues/2690,456200066,MDEyOklzc3VlQ29tbWVudDQ1NjIwMDA2Ng==,5635139,2019-01-21T21:11:57Z,2019-01-21T21:11:57Z,MEMBER,"The reason I had liked this was that I'm often creating a dataset manually, either in a repl or as a fixture in a test, and that added an overhead in both time and reduced consistency.
I recently started using the tutorial data, but that requires a network connection (and I kept forgetting the name...)
But I can empathize re compatibility issues; I defer to @shoyer on the balance, or whether `xarray.testing.dummy_dataset('some_name')` would be feasible","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645
https://github.com/pydata/xarray/pull/2690#issuecomment-456041072,https://api.github.com/repos/pydata/xarray/issues/2690,456041072,MDEyOklzc3VlQ29tbWVudDQ1NjA0MTA3Mg==,1217238,2019-01-21T11:24:56Z,2019-01-21T11:24:56Z,MEMBER,"I think putting an example that can be copied/pasted somewhere on the docs
in the ""extending xarray"" section would be a nice way to solve this.
On Mon, Jan 21, 2019 at 11:18 AM Tom Nicholas
wrote:
> Why not just encourage users to copy/paste the helper function into their
> own codebases?
>
> Just because they aren't going to know it exists unless we put it in a
> public place I suppose?
>
> Happy to do any of those things, I just think that having some kind of
> public function/copyable code example etc is a simple thing for us to do
> that will make it just a bit easier for users who want to build upon xarray.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> , or mute
> the thread
>
> .
>
","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645
https://github.com/pydata/xarray/pull/2690#issuecomment-456039039,https://api.github.com/repos/pydata/xarray/issues/2690,456039039,MDEyOklzc3VlQ29tbWVudDQ1NjAzOTAzOQ==,35968931,2019-01-21T11:17:56Z,2019-01-21T11:17:56Z,MEMBER,"> Why not just encourage users to copy/paste the helper function into their own codebases?
Just because they aren't going to know it exists unless we put it in a public place I suppose?
Happy to do any of those things, I just think that having some kind of public function/copyable code example etc is a simple thing for us to do that will make it just a bit easier for users who want to build upon xarray.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645
https://github.com/pydata/xarray/pull/2690#issuecomment-455826431,https://api.github.com/repos/pydata/xarray/issues/2690,455826431,MDEyOklzc3VlQ29tbWVudDQ1NTgyNjQzMQ==,1217238,2019-01-20T00:21:09Z,2019-01-20T00:21:09Z,MEMBER,"Why not just encourage users to copy/paste the helper function into their own codebases? This seems like a pretty good use case for copy/paste.
Otherwise -- maybe we should think a little more carefully about what to include on the result. Or require explicitly providing a name for the desired dataset structure? E.g., `xarray.testing.dummy_dataset('some_name')`","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645
https://github.com/pydata/xarray/pull/2690#issuecomment-455600050,https://api.github.com/repos/pydata/xarray/issues/2690,455600050,MDEyOklzc3VlQ29tbWVudDQ1NTYwMDA1MA==,35968931,2019-01-18T16:12:25Z,2019-01-18T16:12:25Z,MEMBER,"> it would mean effectively freezing the output of this helper function
Or we could have a public `create_example_data`, and an internal `create_test_data`? That way to begin with `create_example_data` is just an alias to the internal `create_test_data`, but if we end up needing to change the internal one (to test for some new type of common indexing edge case for example) then we can just separate them?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645
https://github.com/pydata/xarray/pull/2690#issuecomment-455599135,https://api.github.com/repos/pydata/xarray/issues/2690,455599135,MDEyOklzc3VlQ29tbWVudDQ1NTU5OTEzNQ==,5635139,2019-01-18T16:09:40Z,2019-01-18T16:09:40Z,MEMBER,"> it would mean effectively freezing the output of this helper function
We could add a `param` or `version` arg so we can make changes in the future, if this is a concern?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645
https://github.com/pydata/xarray/pull/2690#issuecomment-455593215,https://api.github.com/repos/pydata/xarray/issues/2690,455593215,MDEyOklzc3VlQ29tbWVudDQ1NTU5MzIxNQ==,1217238,2019-01-18T15:53:13Z,2019-01-18T15:53:13Z,MEMBER,"I am slightly reluctant about this because it would mean effectively freezing the output of this helper function. I guess that would probably be OK, but the design was never quite fully considered!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,400678645