issues: 1947508727
This data as json
id | node_id | number | title | user | state | locked | assignee | milestone | comments | created_at | updated_at | closed_at | author_association | active_lock_reason | draft | pull_request | body | reactions | performed_via_github_app | state_reason | repo | type |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1947508727 | PR_kwDOAMm_X85dBaso | 8322 | Implementation of rust based cftime | 57914115 | open | 0 | 1 | 2023-10-17T14:00:45Z | 2023-10-17T22:20:31Z | CONTRIBUTOR | 0 | pydata/xarray/pulls/8322 | As discussed in #8302, here is a first attempt to implement There are a lot of tests and I struggle to understand all the processing in Also there are some key differences betwwen Finally, and regardless of this PR, I guess there could be a speed improvement by vectorizing operations by replacing this : https://github.com/pydata/xarray/blob/df0ddaf2e68a6b033b4e39990d7006dc346fcc8c/xarray/coding/times.py#L622-L649 by something like this : We can use numpy instead of list comprehensions. It takes a bit more of memory though. |
{ "url": "https://api.github.com/repos/pydata/xarray/issues/8322/reactions", "total_count": 1, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 1, "eyes": 0 } |
13221727 | pull |