Skip to content

wip: fix address discovery retransmission#674

Draft
divagant-martian wants to merge 1 commit into
mainfrom
send-all-address-discovery-frames
Draft

wip: fix address discovery retransmission#674
divagant-martian wants to merge 1 commit into
mainfrom
send-all-address-discovery-frames

Conversation

@divagant-martian
Copy link
Copy Markdown
Collaborator

@divagant-martian divagant-martian commented May 28, 2026

starting with failing test

@github-actions
Copy link
Copy Markdown

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/noq/pr/674/docs/noq/

Last updated: 2026-05-28T15:43:53Z

@github-actions
Copy link
Copy Markdown

Performance Comparison Report

8602761655a357ac0a7f58d3aa3f49a4e29ff056 - artifacts

Raw Benchmarks (localhost)

Scenario noq upstream Delta CPU (avg/max)
large-single 5498.9 Mbps 7912.4 Mbps -30.5% 96.2% / 148.0%
medium-concurrent 5662.5 Mbps 7910.0 Mbps -28.4% 98.5% / 150.0%
medium-single 3981.3 Mbps 4650.9 Mbps -14.4% 100.0% / 152.0%
small-concurrent 3920.1 Mbps 5388.4 Mbps -27.2% 95.6% / 102.0%
small-single 3611.9 Mbps 4632.0 Mbps -22.0% 94.5% / 102.0%

Netsim Benchmarks (network simulation)

Condition noq upstream Delta
ideal 3142.3 Mbps 3961.9 Mbps -20.7%
lan 782.4 Mbps 816.5 Mbps -4.2%
lossy 69.8 Mbps 69.8 Mbps ~0%
wan 83.8 Mbps 83.8 Mbps ~0%

Summary

noq is 24.5% slower on average

@n0bot n0bot Bot added this to iroh May 28, 2026
@github-project-automation github-project-automation Bot moved this to 🚑 Needs Triage in iroh May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🚑 Needs Triage

Development

Successfully merging this pull request may close these issues.

QAD on Multipath: do we send all OBSERVED_ADDRESS frames for all paths

1 participant