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

cleanup(webconnectivitylte): remove the orig engine #1455

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

bassosimone
Copy link
Contributor

This commit removes the "orig" data analysis engine of the webconnectivitylte experiment now that the "classic" engine is able to generate the same or better results.

Part of ooni/probe#2640

This commit removes the "orig" data analysis engine of the
webconnectivitylte experiment now that the "classic" engine
is able to generate the same or better results.

Part of ooni/probe#2640
@bassosimone bassosimone requested a review from hellais as a code owner January 19, 2024 13:59
@bassosimone bassosimone merged commit b6cfb0a into master Jan 19, 2024
11 checks passed
@bassosimone bassosimone deleted the issue/2640lmn branch January 19, 2024 14:11
bassosimone added a commit that referenced this pull request Jan 22, 2024
This diff ensures that we do not lose any comment or test that we
removed in #1455 and rather aims
to ensure we have equivalent tests and comments.

Part of ooni/probe#2652.
Murphy-OrangeMud pushed a commit to Murphy-OrangeMud/probe-cli that referenced this pull request Feb 13, 2024
This commit removes the "orig" data analysis engine of the
webconnectivitylte experiment now that the "classic" engine is able to
generate the same or better results.

Part of ooni/probe#2640
Murphy-OrangeMud pushed a commit to Murphy-OrangeMud/probe-cli that referenced this pull request Feb 13, 2024
This diff ensures that we do not lose any comment or test that we
removed in ooni#1455 and rather aims
to ensure we have equivalent tests and comments.

Part of ooni/probe#2652.
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