Skip to content

docs: publish measured p-map parity numbers in the migration guide - #403

Open
ppeeou wants to merge 1 commit into
mainfrom
docs/benchmark-numbers
Open

ppeeou wants to merge 1 commit into
mainfrom
docs/benchmark-numbers

Conversation

@ppeeou

@ppeeou ppeeou commented Aug 29, 2026

Copy link
Copy Markdown
Member

Summary

Adds a "Measured" section to the p-limit/p-map migration guide (en/ko/ja/zh) with benchmark numbers taken against the published packages — @fxts/core 2.0.1 vs p-map 7.0.7, Node 24, median of 5 runs per cell, result order/length asserted on every run:

Scenario p-map concurrentPool concurrent
Uneven durations (6 tasks, c=2) 455ms 455ms 654ms
Uniform 50 × 50ms, c=5 510ms 511ms 511ms
5,000 already-resolved tasks, c=10 4ms 9ms 2ms

The framing is deliberately "migrating is not a performance trade" rather than "faster": pool parity is exact on I/O-bound workloads, and the no-op row is kept as honest fine print (~1µs/item order-preserving bookkeeping that vanishes once tasks do real work). The concurrent column doubles as evidence for the guide's existing window-vs-pool warning.

Test plan

  • Numbers reproduced from a 5-repetition median benchmark against npm-published versions
  • vitepress build docs passes; prettier clean; all four locales updated identically

🤖 Generated with Claude Code

Median of 5 runs on the published packages (@fxts/core 2.0.1 vs
p-map 7, Node 24): concurrentPool matches p-map exactly on uneven
(455ms == 455ms) and uniform (511ms vs 510ms) I/O workloads, with the
honest fine print that no-op tasks pay ~1us/item for order-preserving
bookkeeping, and the concurrent column showing window behavior (654ms
on uneven). The framing is deliberately "no performance trade", not
"faster".

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@hg-pyun hg-pyun left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

oh.. claude!

This branch has not been deployed

No deployments
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