◐ Shell
reader mode source ↗
Skip to content

bpo-47189: What's New in 3.11: Faster CPython#32235

Merged
Fidget-Spinner merged 20 commits into
python:mainfrom
Fidget-Spinner:faster_cpython_whatsnew
Apr 6, 2022
Merged

bpo-47189: What's New in 3.11: Faster CPython#32235
Fidget-Spinner merged 20 commits into
python:mainfrom
Fidget-Spinner:faster_cpython_whatsnew

Conversation

@Fidget-Spinner

@Fidget-Spinner Fidget-Spinner commented Apr 1, 2022

Copy link
Copy Markdown
Member

The static objects section's credits go to Kumar Aditya.

TODO:

  • Lazy __dict__ optimizations
  • [ ] dict memory saving optimizations by Inada-san, but I have no clue where to start for those

https://bugs.python.org/issue47189

@bedevere-bot bedevere-bot added the docs Documentation in the Doc dir label Apr 1, 2022
@Fidget-Spinner Fidget-Spinner changed the title What's New in 3.11: Faster CPython Apr 1, 2022
@Fidget-Spinner

Copy link
Copy Markdown
Member Author

CC @gvanrossum @markshannon @brandtbucher @iritkatriel @sweeneyde @corona10 @kumaraditya303 @JelleZijlstra. For your opinions and review please (especially on the parts y'all contributed).

@corona10 corona10 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

Thank you, It is a great honor to see my name on the list.

@markshannon

Copy link
Copy Markdown
Member

Thank you, It is a great honor to see my name on the list.

You deserve it.

@markshannon markshannon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

Thanks for doing this. It is much appreciated.
I've left a few comments.

Fidget-Spinner and others added 2 commits April 3, 2022 14:44
Co-Authored-By: Alex Waygood <Alex.Waygood@Gmail.com>
@Fidget-Spinner

Copy link
Copy Markdown
Member Author

@pablogsal are you OK with this going into 3.11 whats new? Or would you prefer I host this somewhere else and link to it in the whatsnew?

31 hidden items Load more…
@pablogsal

Copy link
Copy Markdown
Member

@pablogsal are you OK with this going into 3.11 whats new? Or would you prefer I host this somewhere else and link to it in the whatsnew?

For the time leave it here in its own section as in this way I will be able to easily do the final edits. Removing or moving information is much easier to add it as that way I don't need to consider what may be missing :)

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

I put my nit-picking hat on while reviewing this. :-) Overall great work, and can't wait to see this show up on docs.python.org!

8 hidden conversations Load more…
Fidget-Spinner and others added 2 commits April 5, 2022 18:54
Co-Authored-By: Guido van Rossum <gvanrossum@users.noreply.github.com>
Co-Authored-By: Irit Katriel <1055913+iritkatriel@users.noreply.github.com>

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

Bunch of one-character nits. :-)

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

Honestly this PR is so large and comments are flying so fast we should probably land it ASAP and then iterate.

Co-Authored-By: Guido van Rossum <gvanrossum@users.noreply.github.com>
@Fidget-Spinner

Fidget-Spinner commented Apr 6, 2022

Copy link
Copy Markdown
Member Author

Also, it's not clear for what operation you are claiming a 170% speedup (so a factor 2.7). The table layout makes it look like C(arg). But surely the speedup for class constructors is not a single extremely large number?

Oof the table layout worked against me. The 170% speedup is tied to "Calls to certain python functions are inlined", not C(arg). I have since removed that line altogether.

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

Thanks, this looks good now. Thanks very much for all your work on this!

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

Labels

docs Documentation in the Doc dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.