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

Implement the fast path for list.__getitem__ #119053

Closed
Tracked by #108219
mpage opened this issue May 14, 2024 · 4 comments
Closed
Tracked by #108219

Implement the fast path for list.__getitem__ #119053

mpage opened this issue May 14, 2024 · 4 comments
Assignees
@corona10
Labels
performance Performance or resource usage topic-free-threading type-feature A feature request or enhancement

Comments

@mpage
Copy link
Contributor

mpage commented May 14, 2024 •
edited by bedevere-app bot

Feature or enhancement

Proposal:

Implement the fast path for list.__getitem__ outlined in PEP-703.

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

No response

Linked PRs
@mpage mpage added type-feature A feature request or enhancement topic-free-threading labels May 14, 2024
@mpage mpage mentioned this issue May 14, 2024
Open
@mpage mpage added the performance Performance or resource usage label May 14, 2024
@corona10
Copy link
Member

@mpage Can I take a look at this one or you already work on this issue? (I am just asking to avoid the confliction)

corona10 added a commit to corona10/cpython that referenced this issue May 17, 2024
@corona10
pythongh-119053: Implement the fast path for list.__getitem__
f2cd85c
@bedevere-app bedevere-app bot mentioned this issue May 17, 2024
Merged
corona10 added a commit to corona10/cpython that referenced this issue May 17, 2024
@corona10
pythongh-119053: Implement the fast path for list.__getitem__
5fb201d
corona10 added a commit to corona10/cpython that referenced this issue May 17, 2024
@corona10
pythongh-119053: Implement the fast path for list.__getitem__
f895ca9
@mpage
Copy link
Contributor Author

mpage commented May 17, 2024

@corona10 - Go for it!

corona10 added a commit to corona10/cpython that referenced this issue May 18, 2024
@corona10
Merge remote-tracking branch 'upstream/main' into pythongh-119053
0e22983
corona10 added a commit that referenced this issue May 21, 2024
@corona10
gh-119053: Implement the fast path for list.__getitem__ (gh-119112)
ab4263a
miss-islington pushed a commit to miss-islington/cpython that referenced this issue May 21, 2024
@corona10 @miss-islington
pythongh-119053: Implement the fast path for list.__getitem__ (python…
11f687f 
…gh-119112)

(cherry picked from commit ab4263a)

Co-authored-by: Donghee Na <donghee.na@python.org>
@bedevere-app bedevere-app bot mentioned this issue May 21, 2024
Merged
corona10 added a commit that referenced this issue May 21, 2024
@miss-islington @corona10
[3.13] gh-119053: Implement the fast path for list.__getitem__ (gh-11…
dcb8030 
…9112) (gh-119309)

gh-119053: Implement the fast path for list.__getitem__ (gh-119112)
(cherry picked from commit ab4263a)

Co-authored-by: Donghee Na <donghee.na@python.org>
@JelleZijlstra JelleZijlstra mentioned this issue May 28, 2024
Closed
@nineteendo
Copy link
Contributor

Is there still something to do here?

@colesbury
Copy link
Contributor

No, it's done thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@corona10 corona10

Labels
performance Performance or resource usage topic-free-threading type-feature A feature request or enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mpage @colesbury @corona10 @nineteendo