-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Full MDA implementation #4
base: main
Are you sure you want to change the base?
Full MDA implementation #4
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4 +/- ##
===========================================
- Coverage 98.98% 88.19% -10.80%
===========================================
Files 17 18 +1
Lines 297 398 +101
===========================================
+ Hits 294 351 +57
- Misses 3 47 +44 ☔ View full report in Codecov by Sentry. |
@maxmouchet @timur-friedman Consequently, a lot of tests were added (100 at this time), using real-life topologies and artificial ones, like meshed networks. diamond-miner is parametrised with a 95% confidence rate, and I check that topologies are discovered more than 75% of the time. The difference is due to the confidence rate being a local feature of next-hop discovery and topology recovery rate is a global feature. Since the link between these two measure is not immediately clear to me, I settled on the 95/75 metrics for now, though this will need to be revisited for validation. |
Summary of changes
Context
After multiple experiments comparing performance of fmda vs scamper, some issues appeared relative to the number of discovered IPs and the number of probes sent, among others.
I was tasked by @timur-friedman to investigate and modify fmda if necessary. Since I'm no network expert, I would be extremely grateful for a careful review of the full mda part. The suggested changes were quantitatively evaluated on thousands of paths vs. scamper.
In the near future however, fmda will be entirely rewritten in Rust (port has already started). The Python version should help network measurements actors to evaluate the precision and gains of fmda, before switching to a full Rust version that benefits from easier packaging, thanks to work on caracat.
As a result, I did not edit the RATIONALE.md file, nor did I polish additional options or tests for now.
Do not hesitate to ping me for any clarification.