pull_requests: 1839886950
This data as json
id | node_id | number | state | locked | title | user | body | created_at | updated_at | closed_at | merged_at | merge_commit_sha | assignee | milestone | draft | head | base | author_association | auto_merge | repo | url | merged_by |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1839886950 | PR_kwDOAMm_X85tqnJm | 8973 | closed | 0 | Docstring and documentation improvement for the Dataset class | 2005723 | The example in the doc-string of the `Dataset` class prior to this commit uses an example array whose size is `2 x 2 x 3` with the first two dimensions labeled `"x"` and `"y"` and the final dimension labeled `"time"`. This was confusing due to the fact that `"x"` and `"y"` are just arbitrary names for these axes and that no reason is given for the data to be organized in a `2x2x3` array instead of a `2x2` matrix. This commit clarifies the example. Additionally, this PR contains updates to the documentation, specifically the `user-guide/data-structures.rst` file; the updates bring the documentation examples into alignment with the doc-string change. Unfortunately, I wasn't able to build the documentation, so this will need to be checked. (I followed the instructions [here](https://docs.xarray.dev/en/stable/contributing.html#contributing-to-the-documentation), but despite `cfgrib` working fine, I got an error about how it wasn't a valid engine.) See issue #8970 for more information. <!-- Feel free to remove check-list items aren't relevant to your change --> - [X] Closes #8970 | 2024-04-25T01:39:02Z | 2024-04-30T14:40:32Z | 2024-04-30T14:40:14Z | 2024-04-30T14:40:14Z | 71372c1bfcdf2de7c15c0ef8ab2315b7f482497b | 0 | b92c52dd8a90563767c0fb6004e194fa13ed5b83 | 08e43b9121a6ae1f59e5c6b84caa327d816ea777 | CONTRIBUTOR | 13221727 | https://github.com/pydata/xarray/pull/8973 |
Links from other tables
- 1 row from pull_requests_id in labels_pull_requests