Skip to content
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

Ndt test/description #80

Merged
merged 3 commits into from
Nov 12, 2016
Merged

Ndt test/description #80

merged 3 commits into from
Nov 12, 2016

Conversation

agrabeli
Copy link
Member

This is a description for the NDT test that is included in ooni-mobile. We can publish this description under the Nettests section of the website once OONI's mobile app has been released.

*performance* of tested networks.

This network performance test was originally developed by The Internet2 Project
and is currently maintained by [Measurement Lab (mLab)](http://www.measurementlab.net/tools/ndt/). NDT is designed to measure
Copy link
Contributor

Choose a reason for hiding this comment

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

The canonical short name for Measurement Lab is M-Lab

tests. This NDT implementation, for example, is included as a test that can be
run via OONI's mobile app.

As OONI's core software tests are *not* designed to collect PCAPs or low level
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe in the future we could change whether PCAPs are collected or not, perhaps I'd phrase the sentence to say just that the information collected by NDT can be potentially used to examine cases of throttling

Copy link
Contributor

@bassosimone bassosimone left a comment

Choose a reason for hiding this comment

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

Looks good to me, except a couple of cases for which I commented inline. Thank you!

@agrabeli
Copy link
Member Author

@bassosimone Thanks, I've implemented your feedback.

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.

3 participants