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