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

doc: explain the throttling methodology #1546

Merged
merged 18 commits into from
Jun 21, 2024
Merged

doc: explain the throttling methodology #1546

merged 18 commits into from
Jun 21, 2024

Conversation

bassosimone
Copy link
Contributor

@bassosimone bassosimone commented Apr 9, 2024

  • outline the current methodology;

  • explain its theoretical assumptions;

  • describe limitations;

  • mention possible future directions.

Part of ooni/ooni.org#1406.

Closes #684 because we've now gone full circle and we have infrastructure to setup a comparable experiment via richer input without the need of implementing a new experiment.

- outline the current methodology;

- explain its theoretical assumptions;

- describe limitations;

- mention possible future directions.

Part of ooni/ooni.org#1406.
@bassosimone bassosimone self-assigned this Apr 9, 2024
@bassosimone bassosimone requested a review from DecFox April 9, 2024 16:44
Copy link
Contributor

@DecFox DecFox left a comment

Choose a reason for hiding this comment

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

As discussed with @bassosimone, we should also document the following:

  • how we can use this methodology while looking at data from webconnectivity v0.5: analyzing speeds between domains across different temporal data points.

Apart from this, the document looks good and is well-explained. This serves as a good foundation to analyse throttling incidents.

Additionally, mention comments that were provided by @agrabeli,
@albertodainotti, et al., during a recent gathering focusing on
network measurements where I presented this work.
@bassosimone bassosimone requested a review from DecFox June 21, 2024 09:25
Copy link
Contributor

@DecFox DecFox left a comment

Choose a reason for hiding this comment

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

The changes look good. I left a small comment. I think this should be good to go

docs/design/dd-007-throttling.md Outdated Show resolved Hide resolved
Co-authored-by: DecFox <33030671+DecFox@users.noreply.github.com>
@bassosimone bassosimone merged commit f10bad6 into master Jun 21, 2024
17 checks passed
@bassosimone bassosimone deleted the issue/1406 branch June 21, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants