Skip to content

Add synthesize_loops_file script - #395

Merged
mdboom merged 3 commits into
python:mainfrom
Yhg1s:synthesize_loops_script
Apr 9, 2025
Merged

mdboom merged 3 commits into
python:mainfrom
Yhg1s:synthesize_loops_script

Conversation

@Yhg1s

@Yhg1s Yhg1s commented Apr 5, 2025

Copy link
Copy Markdown
Member

Add a command to synthesize a pyperformance loops file from one or more benchmark results.

@mdboom
mdboom marked this pull request as draft April 7, 2025 13:05
Yhg1s added 2 commits April 7, 2025 15:20
data points and benchmarks, and drop the mean/median options for merging
results, as they've become even more nonsensical now.
@Yhg1s
Yhg1s force-pushed the synthesize_loops_script branch from 8aed397 to 070e1ec Compare April 7, 2025 13:20
@mdboom
mdboom marked this pull request as ready for review April 9, 2025 21:31

@mdboom mdboom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks fine, and mergeable (it's independent from everything else, anyway).

Let's together work on fixing the issues upstream in pyperformance that make these hard-coded lists necessary and then remove them when that's done.

@mdboom
mdboom merged commit 2147aad into python:main Apr 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants