X Tutup
Skip to content

GH-142267: Cache formatter to avoid repeated _set_color calls#142268

Merged
savannahostrowski merged 7 commits intopython:mainfrom
savannahostrowski:argparse-color-perf
Dec 5, 2025
Merged

GH-142267: Cache formatter to avoid repeated _set_color calls#142268
savannahostrowski merged 7 commits intopython:mainfrom
savannahostrowski:argparse-color-perf

Conversation

@savannahostrowski
Copy link
Member

@savannahostrowski savannahostrowski commented Dec 4, 2025 •
edited
Loading

bm_subparsers benchmark (1000 optional arguments, 10 iterations):

Version Time
3.13 0.246s
3.14 0.934s
3.15 (main) 0.883s
3.15 (with fix) 0.158s

Profiled with Tachyon (./python.exe -m profiling.sampling --live):

Before:
🥇 HelpFormatter._set_color (12.6%) │ 🥈 Mapping.get (5.4%) │ 🥉 can_colorize._safe_getenv (4.1%)

After:
🥇 <GC> (8.4%) │ 🥈 bm_subparsers (8.3%) │ 🥉 _ActionsContainer.add_argument (7.3%)

@savannahostrowski savannahostrowski added the needs backport to 3.14 bugs and security fixes label Dec 4, 2025
@bedevere-app bedevere-app bot mentioned this pull request Dec 4, 2025
Closed
@savannahostrowski savannahostrowski mentioned this pull request Dec 4, 2025
Closed
serhiy-storchaka
serhiy-storchaka reviewed Dec 5, 2025
View reviewed changes
vstinner
vstinner reviewed Dec 5, 2025
View reviewed changes
serhiy-storchaka
serhiy-storchaka approved these changes Dec 5, 2025
View reviewed changes
Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

LGTM. 👍

hugovk
hugovk approved these changes Dec 5, 2025
View reviewed changes
Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

Thanks!

@savannahostrowski savannahostrowski enabled auto-merge (squash) December 5, 2025 16:20
@savannahostrowski savannahostrowski merged commit 4085ff7 into python:main Dec 5, 2025
46 checks passed
@miss-islington-app
Copy link

miss-islington-app bot commented Dec 5, 2025

Thanks @savannahostrowski for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 5, 2025
@savannahostrowski @miss-islington
pythonGH-142267: Cache formatter to avoid repeated _set_color calls (…
146f494 
…pythonGH-142268)

(cherry picked from commit 4085ff7)

Co-authored-by: Savannah Ostrowski <savannah@python.org>
@bedevere-app
Copy link

bedevere-app bot commented Dec 5, 2025

GH-142313 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Dec 5, 2025
savannahostrowski added a commit that referenced this pull request Dec 5, 2025
@miss-islington @savannahostrowski
[3.14] GH-142267: Cache formatter to avoid repeated _set_color calls (
37c6dd1 
GH-142268) (#142313)

GH-142267: Cache formatter to avoid repeated `_set_color` calls (GH-142268)
(cherry picked from commit 4085ff7)

Co-authored-by: Savannah Ostrowski <savannah@python.org>
StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
@savannahostrowski @StanFromIreland
pythonGH-142267: Cache formatter to avoid repeated _set_color calls (…
badffd6 
…python#142268)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinner vstinner vstinner left review comments

@hugovk hugovk hugovk approved these changes

@serhiy-storchaka serhiy-storchaka serhiy-storchaka approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@savannahostrowski @vstinner @hugovk @serhiy-storchaka
X Tutup