pull_requests: 1145695726

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
1145695726 PR_kwDOAMm_X85ESe3u 7356 closed 0 Avoid loading entire dataset by getting the nbytes in an array 90008 Using `.data` accidentally tries to load the whole lazy arrays into memory. Sad. <!-- Feel free to remove check-list items aren't relevant to your change --> - [ ] Closes #xxxx - [ ] Tests added - [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst` - [ ] New functions/methods are listed in `api.rst` 2022-12-05T03:29:53Z 2023-03-17T17:31:22Z 2022-12-12T16:46:40Z 2022-12-12T16:46:40Z 021c73e12cccb06c017ce6420dd043a0cfbf9f08     0 02e3cb10c2d49569b888532f48ba8e47226c1e85 db68db6793bdd10f740e1ff7f68d821e853e3d73 CONTRIBUTOR 
{
    "enabled_by": {
        "login": "dcherian",
        "id": 2448579,
        "node_id": "MDQ6VXNlcjI0NDg1Nzk=",
        "avatar_url": "https://avatars.githubusercontent.com/u/2448579?v=4",
        "gravatar_id": "",
        "url": "https://api.github.com/users/dcherian",
        "html_url": "https://github.com/dcherian",
        "followers_url": "https://api.github.com/users/dcherian/followers",
        "following_url": "https://api.github.com/users/dcherian/following{/other_user}",
        "gists_url": "https://api.github.com/users/dcherian/gists{/gist_id}",
        "starred_url": "https://api.github.com/users/dcherian/starred{/owner}{/repo}",
        "subscriptions_url": "https://api.github.com/users/dcherian/subscriptions",
        "organizations_url": "https://api.github.com/users/dcherian/orgs",
        "repos_url": "https://api.github.com/users/dcherian/repos",
        "events_url": "https://api.github.com/users/dcherian/events{/privacy}",
        "received_events_url": "https://api.github.com/users/dcherian/received_events",
        "type": "User",
        "site_admin": false
    },
    "merge_method": "squash",
    "commit_title": "Avoid loading entire dataset by getting the nbytes in an array (#7356)",
    "commit_message": "* Avoid instantiating entire dataset by getting the nbytes in an array\r\n\r\nUsing `.data` accidentally tries to load the whole lazy arrays into\r\nmemory.\r\n\r\nSad.\r\n\r\n* DOC: Add release note for bugfix.\r\n\r\n* Add test to ensure that number of bytes of sparse array is correctly\r\nreported\r\n\r\n* Add suggested test using InaccessibleArray\r\n\r\n* [pre-commit.ci] auto fixes from pre-commit.com hooks\r\n\r\nfor more information, see https://pre-commit.ci\r\n\r\n* Remove duplicate test\r\n\r\nCo-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>\r\nCo-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>"
}
 13221727 https://github.com/pydata/xarray/pull/7356  

Links from other tables