Skip to content

More perf improvements#638

Open
dcherian wants to merge 3 commits intomainfrom
perf-getitem-hot-path
Open

More perf improvements#638
dcherian wants to merge 3 commits intomainfrom
perf-getitem-hot-path

Conversation

@dcherian
Copy link
Copy Markdown
Contributor

@dcherian dcherian commented May 2, 2026

No description provided.

dcherian and others added 3 commits May 1, 2026 20:25
The string is typically shared across all variables in a dataset, so
parse once per distinct attr. Wrap the cached result in `Frozen` to
prevent accidental mutation of the shared value.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Skip the per-key _get_all fan-out when every key is already a plain
data-variable name — the slow path would just fall through to the same
result. Decoration via get_associated_variable_names is unchanged.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
drop_bounds is a no-op unless the caller passed a scalar key, so
list-key calls (ds.cf[[name]]) never need it. Inline the lookup so
list-key paths skip the full-dataset bounds scan entirely.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant