home / github / pull_requests

Menu
  • Search all tables
  • GraphQL API

pull_requests: 1032111288

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
1032111288 PR_kwDOAMm_X849hMS4 6940 closed 0 Enable taking the mean of dask-backed cftime arrays 6628425 This was essentially enabled by @dcherian in #6556, but we did not remove the error that prevented computing the mean of a dask-backed cftime array. This PR removes that error, and adds some tests. One minor modification in `_timedelta_to_seconds` was needed for compatibility with scalar cftime arrays. This happens to address the second part of #5897, so I added a regression test for that. It seems like we decided to simply document the behavior in the first part (https://github.com/pydata/xarray/issues/5898, https://github.com/dcherian/xarray/commit/99bfe128066ec3ef1b297650a47e2dd0a45801a8), but I'm not sure if we intend to change that behavior eventually or not. - [x] Tests added - [x] User visible changes (including notable bug fixes) are documented in `whats-new.rst` 2022-08-21T19:24:37Z 2022-09-10T12:28:16Z 2022-09-09T16:48:19Z 2022-09-09T16:48:19Z 25537623edafd4a2f99a011ebb91ae55bccb96a2     0 caf11162e0b9c99948cd2e33694adf84d7896fc3 abe1e613a96b000ae603c53d135828df532b952e MEMBER
{
    "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": "Enable taking the mean of dask-backed cftime arrays (#6940)",
    "commit_message": "Co-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/6940  

Links from other tables

  • 4 rows from pull_requests_id in labels_pull_requests
Powered by Datasette · Queries took 2.534ms