home / github / pull_requests

Menu
  • GraphQL API
  • Search all tables

pull_requests: 1558999807

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
1558999807 PR_kwDOAMm_X85c7HL_ 8319 closed 0 Move parallelcompat and chunkmanagers to NamedArray 35968931 @dcherian I got to this point before realizing that simply moving `parallelcompat.py` over isn't [what it says in the design doc](https://github.com/pydata/xarray/blob/main/design_notes/named_array_design_doc.md#appendix-implementation-details), which instead talks about > - Could this functionality be left in Xarray proper for now? Alternative array types like JAX also have some notion of "chunks" for parallel arrays, but the details differ in a number of ways from the Dask/Cubed. > - Perhaps variable.chunk/load methods should become functions defined in xarray that convert Variable objects. This is easy so long as xarray can reach in and replace `.data` I personally think that simply moving parallelcompat makes sense so long as you expect people to use chunked `NamedArray` objects. I see the chunked arrays as special cases of duck arrays, and my understanding is that `NamedArray` is supposed to have full support for duckarrays. cc @andersy005 - [x] As requested in #8238 - [ ] ~~Tests added~~ - [ ] ~~User visible changes (including notable bug fixes) are documented in `whats-new.rst`~~ - [ ] ~~New functions/methods are listed in `api.rst`~~ 2023-10-16T16:34:26Z 2024-02-12T22:09:24Z 2024-02-12T22:09:24Z 2024-02-12T22:09:24Z d64460795e406bc4a998e2ddae0054a1029d52a9     0 ea7feef45437b2f0b8a97aae61d4685cd7ee6eb9 3d490ec3f1cd0ac9114a5d270550ce6e7864ade7 MEMBER
{
    "enabled_by": {
        "login": "andersy005",
        "id": 13301940,
        "node_id": "MDQ6VXNlcjEzMzAxOTQw",
        "avatar_url": "https://avatars.githubusercontent.com/u/13301940?v=4",
        "gravatar_id": "",
        "url": "https://api.github.com/users/andersy005",
        "html_url": "https://github.com/andersy005",
        "followers_url": "https://api.github.com/users/andersy005/followers",
        "following_url": "https://api.github.com/users/andersy005/following{/other_user}",
        "gists_url": "https://api.github.com/users/andersy005/gists{/gist_id}",
        "starred_url": "https://api.github.com/users/andersy005/starred{/owner}{/repo}",
        "subscriptions_url": "https://api.github.com/users/andersy005/subscriptions",
        "organizations_url": "https://api.github.com/users/andersy005/orgs",
        "repos_url": "https://api.github.com/users/andersy005/repos",
        "events_url": "https://api.github.com/users/andersy005/events{/privacy}",
        "received_events_url": "https://api.github.com/users/andersy005/received_events",
        "type": "User",
        "site_admin": false
    },
    "merge_method": "squash",
    "commit_title": "Move parallelcompat and chunkmanagers to NamedArray (#8319)",
    "commit_message": "Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>\r\nCo-authored-by: Anderson Banihirwe <13301940+andersy005@users.noreply.github.com>\r\nCo-authored-by: Anderson Banihirwe <axbanihirwe@ualr.edu>\r\n"
}
13221727 https://github.com/pydata/xarray/pull/8319  

Links from other tables

  • 15 rows from pull_requests_id in labels_pull_requests
Powered by Datasette · Queries took 0.756ms