Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Deprecate inclusive range in MemoryReader #3893

Open
hmacdope opened this issue Oct 29, 2022 · 0 comments
Open

Deprecate inclusive range in MemoryReader #3893

hmacdope opened this issue Oct 29, 2022 · 0 comments
Assignees
@hmacdope
Labels
deprecation Deprecated functionality to give advance warning for API changes. Format-MemoryReader
Milestone
Release 3.0

Comments

@hmacdope
Copy link
Member

Expected behavior

The MemoryReader should use non-inclusive indexing like Python's range and Numpy.

Actual behavior

The memoryreader is documented as having inclusive ranges

Following discussion in #3890 we would like to deprecate this behaviour and change in 3.0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hmacdope hmacdope

Labels
deprecation Deprecated functionality to give advance warning for API changes. Format-MemoryReader
Projects
CZI Performance track
Status: In Progress
Milestone
Release 3.0
Development

No branches or pull requests
1 participant
@hmacdope