Skip to content

pooley182#124

Open
pooley182 wants to merge 2 commits intotempestphp:mainfrom
pooley182:main
Open

pooley182#124
pooley182 wants to merge 2 commits intotempestphp:mainfrom
pooley182:main

Conversation

@pooley182
Copy link

I see you've changed the hardware to a mac mini. So re-opening my PR.

Please add Single Threaded tag.

@pooley182
Copy link
Author

/bench

@brendt
Copy link
Member

brendt commented Feb 26, 2026

Benchmarking complete! Mean execution time: 38.99186295462s

@pooley182
Copy link
Author

@xHeaven or @brendt Could you add the single thread tag, please?

@pooley182 pooley182 changed the title single threaded parsing pooley182 Feb 26, 2026
@pooley182
Copy link
Author

I just realised I've messed up your leaderboard by not naming my branch correctly too.
Apologies!

@xHeaven
Copy link
Member

xHeaven commented Feb 26, 2026

I just realised I've messed up your leaderboard by not naming my branch correctly too. Apologies!

Fixed it, no worries.

@pooley182
Copy link
Author

I just realised I've messed up your leaderboard by not naming my branch correctly too. Apologies!

Fixed it, no worries.

Thank you!
I now do appear in the leaderboard twice though 😬

@brendt
Copy link
Member

brendt commented Feb 26, 2026

Benchmarking complete! Mean execution time: 39.25891952308s

@brendt
Copy link
Member

brendt commented Feb 26, 2026

Mean time goes down, confidence goes up

@brendt brendt removed the verified label Feb 26, 2026
Swap file_put_contents for fwrite.
@pooley182
Copy link
Author

/bench

@brendt
Copy link
Member

brendt commented Feb 26, 2026

Benchmarking complete! Mean execution time: 39.21425521818s

@brendt
Copy link
Member

brendt commented Feb 26, 2026

Mean time goes down, confidence goes up

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants