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

websteps: use the same names as webconnectivity: queries, request, etc #1817

Closed
bassosimone opened this issue Oct 14, 2021 · 1 comment
Closed
Assignees
Labels
bug Something isn't working correctly

Comments

@bassosimone
Copy link
Contributor

This simplifies processing and extracting fingerprints.

@bassosimone bassosimone added bug Something isn't working correctly triage labels Oct 14, 2021
@bassosimone bassosimone added this to the Sprint 50 - Amphipoda milestone Oct 14, 2021
@bassosimone bassosimone self-assigned this Oct 14, 2021
@bassosimone bassosimone removed this from the Sprint 50 - Amphipoda milestone Oct 22, 2021
bassosimone added a commit to ooni/probe-cli that referenced this issue Nov 4, 2021
This change should simplify the pipeline's job.

Reference issue: ooni/probe#1817.

I previously dismissed this possibility, but now it seems clear it
is simpler to have a very tabular data format internally and to
convert such a format to OONI's data format when serializing.

The OONI data format is what the pipeline expects, but processing
is easier with a more linear/tabular format.
bassosimone added a commit to ooni/probe-cli that referenced this issue Nov 5, 2021
This change should simplify the pipeline's job.

Reference issue: ooni/probe#1817.

I previously dismissed this possibility, but now it seems clear it
is simpler to have a very tabular data format internally and to
convert such a format to OONI's data format when serializing.

The OONI data format is what the pipeline expects, but processing
is easier with a more linear/tabular format.
@bassosimone
Copy link
Contributor Author

Done in ooni/probe-cli#572

@bassosimone bassosimone removed the triage label Nov 5, 2021
ainghazal pushed a commit to ainghazal/probe-cli that referenced this issue Mar 8, 2022
This change should simplify the pipeline's job.

Reference issue: ooni/probe#1817.

I previously dismissed this possibility, but now it seems clear it
is simpler to have a very tabular data format internally and to
convert such a format to OONI's data format when serializing.

The OONI data format is what the pipeline expects, but processing
is easier with a more linear/tabular format.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working correctly
Projects
None yet
Development

No branches or pull requests

1 participant