issues: 1731320789
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 |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 1731320789 | PR_kwDOAMm_X85Rougi | 7883 | Avoid one call to len when getting ndim of Variables | 90008 | closed | 0 | 3 | 2023-05-29T23:37:10Z | 2023-07-03T15:44:32Z | 2023-07-03T15:44:31Z | CONTRIBUTOR | 0 | pydata/xarray/pulls/7883 | I admit this is a super micro optimization but it avoids in certain cases the creation of a tuple, and a call to len on it. I hit this as I was trying to understand why Variable indexing was so much slower than numpy indexing. It seems that bounds checking in python is just slower than in C. Feel free to close this one if you don't want this kind of optimization.
|
{
"url": "https://api.github.com/repos/pydata/xarray/issues/7883/reactions",
"total_count": 1,
"+1": 1,
"-1": 0,
"laugh": 0,
"hooray": 0,
"confused": 0,
"heart": 0,
"rocket": 0,
"eyes": 0
} |
13221727 | pull |