pull_requests: 1062097375
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 |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1062097375 | PR_kwDOAMm_X84_TlHf | 7060 | closed | 0 | More informative error for non-existent zarr store | 39069044 | - [x] Closes #6484 - [x] Tests added I've often been tripped up by the stack trace noted in #6484. This PR changes two things: 1. Handles the zarr `GroupNotFoundError` error with a more informative `FileNotFoundError`, displaying the path where we didn't find a zarr store. 2. Moves the consolidated metadata warning to after the step of successfully opening the zarr with non-consolidated metadata. This way the warning isn't shown if we are actually trying to open a non-existent zarr store, in which case we only get the error above and no warning. | 2022-09-20T21:27:35Z | 2022-09-20T22:38:45Z | 2022-09-20T22:38:45Z | 2022-09-20T22:38:45Z | e6791852aa7ec0b126048b0986e205e158ab9601 | 0 | a4efa689607d6229d9748d4470bfa8425dd740fe | 716973e41060184beebd64935afe196a805ef481 | CONTRIBUTOR | { "enabled_by": { "login": "max-sixty", "id": 5635139, "node_id": "MDQ6VXNlcjU2MzUxMzk=", "avatar_url": "https://avatars.githubusercontent.com/u/5635139?v=4", "gravatar_id": "", "url": "https://api.github.com/users/max-sixty", "html_url": "https://github.com/max-sixty", "followers_url": "https://api.github.com/users/max-sixty/followers", "following_url": "https://api.github.com/users/max-sixty/following{/other_user}", "gists_url": "https://api.github.com/users/max-sixty/gists{/gist_id}", "starred_url": "https://api.github.com/users/max-sixty/starred{/owner}{/repo}", "subscriptions_url": "https://api.github.com/users/max-sixty/subscriptions", "organizations_url": "https://api.github.com/users/max-sixty/orgs", "repos_url": "https://api.github.com/users/max-sixty/repos", "events_url": "https://api.github.com/users/max-sixty/events{/privacy}", "received_events_url": "https://api.github.com/users/max-sixty/received_events", "type": "User", "site_admin": false }, "merge_method": "squash", "commit_title": "More informative error for non-existent zarr store (#7060)", "commit_message": "* more informative error for non-existent zarr store\r\n\r\n* add whats-new" } |
13221727 | https://github.com/pydata/xarray/pull/7060 |
Links from other tables
- 3 rows from pull_requests_id in labels_pull_requests