Skip to content

PEP 831: Frame pointers #4896

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

Open
pablogsal wants to merge 22 commits into
base: main
Choose a base branch
from
Open

PEP 831: Frame pointers #4896

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
23ea4a5
Frame pointers frame pointers frame pointers
pablogsal Apr 6, 2026
984672f
First pass at edits
savannahostrowski Apr 8, 2026
42c762a
Add ref back
savannahostrowski Apr 8, 2026
2025921
Fix flame graph
savannahostrowski Apr 8, 2026
3408a2b
Fixed whitespace
savannahostrowski Apr 8, 2026
4569c5c
Update preamble
savannahostrowski Apr 8, 2026
6c3bde4
Add RPi and M3 results
savannahostrowski Apr 9, 2026
87a3af7
Clean up table
savannahostrowski Apr 9, 2026
2f241a5
Add my benchmarks
Fidget-Spinner Apr 9, 2026
941880e
Add perf figures, tighten JIT claims
Fidget-Spinner Apr 9, 2026
009204a
remove Diego (for now)
Fidget-Spinner Apr 9, 2026
bd948d0
fix typo
Fidget-Spinner Apr 9, 2026
02c94e6
Show outliers
Fidget-Spinner Apr 9, 2026
1c19495
Update pep-0830.rst
Fidget-Spinner Apr 9, 2026
957cecb
Small fixes
pablogsal Apr 9, 2026
e7faa8c
Address PR feedback
pablogsal Apr 9, 2026
b231f21
Update new results, add alt text
Fidget-Spinner Apr 10, 2026
9360134
exclude svg from pre-commit
Fidget-Spinner Apr 10, 2026
c7269af
Update .pre-commit-config.yaml
Fidget-Spinner Apr 10, 2026
42dcac9
Merge pull request #9 from Fidget-Spinner/fp-kj-3
pablogsal Apr 10, 2026
df300e1
Apply suggestions from code review
pablogsal Apr 11, 2026
3b8af0a
Apply suggestions
pablogsal Apr 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/CODEOWNERS
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -703,6 +703,7 @@ peps/pep-0826.rst @savannahostrowski
peps/pep-0827.rst @1st1
peps/pep-0828.rst @ZeroIntensity
peps/pep-0829.rst @warsaw
peps/pep-0831.rst @pablogsal @Fidget-Spinner @savannahostrowski
# ...
peps/pep-2026.rst @hugovk
# ...
Expand Down
1 change: 1 addition & 0 deletions .pre-commit-config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ repos:
args: [--fix=lf]
- id: trailing-whitespace
name: "Remove trailing whitespace"
exclude: .*\.svg
Copy link
Copy Markdown
Member

@hugovk hugovk Apr 11, 2026

Choose a reason for hiding this comment

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

@Fidget-Spinner Did this break the SVGs? They're code, so it's usually okay to allow this.

Suggested change
exclude: .*\.svg

Copy link
Copy Markdown
Member

@Fidget-Spinner Fidget-Spinner Apr 11, 2026

Choose a reason for hiding this comment

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

Yes they break the svgs

Copy link
Copy Markdown
Member

@Fidget-Spinner Fidget-Spinner Apr 11, 2026

Choose a reason for hiding this comment

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

Wait let me correct, I'm not sure if they break it but its very disruptive to reformat svgs

Copy link
Copy Markdown
Member

@hugovk hugovk Apr 11, 2026

Choose a reason for hiding this comment

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

How disruptive? Is it just a one-off automated thing, and then commit and carry one?

If it's a blocker to remove the trailing spaces, please only exclude PEP 831 SVGs.


- id: file-contents-sorter
name: "Sort codespell ignore list"
Expand Down
Loading
Loading