Skip to content

feat: add multi-target bandwidth measurements - #154

Open
andre-j3sus wants to merge 3 commits into
mainfrom
ajesus/add-parallel-bandwidth-hosts
Open

feat: add multi-target bandwidth measurements#154
andre-j3sus wants to merge 3 commits into
mainfrom
ajesus/add-parallel-bandwidth-hosts

Conversation

@andre-j3sus

@andre-j3sus andre-j3sus commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Adds measurementTargets as the origin-based contract for latency, download, and upload distribution. Existing full download and upload URLs remain as deprecated fallbacks when no targets are configured. Sequential and parallel requests use randomized-start target ordering, preserving duplicate targets.

Bandwidth steps can opt into experimental parallel execution through the TypeScript API. Parallel steps maintain one continuously replenished lane per selected target, keep count as the total physical request count, and report one aggregate point spanning the complete step. Aggregate duration excludes overlapping server-processing adjustments and paused intervals. Per-request logging remains physical-request based, while session metadata records the configured target count only when parallel execution is enabled.

@andre-j3sus andre-j3sus self-assigned this Aug 28, 2026
@ask-bonk

ask-bonk Bot commented Aug 28, 2026

Copy link
Copy Markdown

LGTM!

github run

Comment thread README.md Outdated
@andre-j3sus
andre-j3sus marked this pull request as ready for review September 1, 2026 16:12
@andre-j3sus andre-j3sus changed the title feat: add parallel bandwidth measurements feat: add multi-target bandwidth measurements Sep 2, 2026
@andre-j3sus

Copy link
Copy Markdown
Collaborator Author

/bonk

Comment thread src/engines/BandwidthEngine/BandwidthEngine.ts Outdated
@ask-bonk

ask-bonk Bot commented Sep 2, 2026

Copy link
Copy Markdown

Adds target-based request distribution, experimental per-step parallel bandwidth execution, aggregate result reporting, and per-request logging.

Posted 1 inline correctness finding.

Checks: git diff --check passed. Unit tests could not run because pnpm is unavailable.

github run

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.

1 participant