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.

Sign up for GitHub

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

Jump to bottom

mobile: use Go API for running experiments #1346

Closed
bassosimone opened this issue Feb 9, 2021 · 3 comments
Closed

mobile: use Go API for running experiments #1346

bassosimone opened this issue Feb 9, 2021 · 3 comments
Assignees
@bassosimone
Labels
priority/high Important issue that needs attention soon
Milestone
Sprint 35 - Octar...

Comments

@bassosimone
Copy link
Contributor

Yesterday we had a team meeting where we discussed how we want to use a Go API to run experiments. This will allow us to significantly cut the complexity of marshaling messages to/from JSON. We can afford that, now that we removed MK.

We should write a simple wrapping API that works on mobile and allows us to run all the experiments. We don't need to do this for all experiments together. Starting with one-two experiments allows to to evaluate what to do on the mobile side.
@bassosimone bassosimone added priority/high Important issue that needs attention soon effort/XL labels Feb 9, 2021
@bassosimone bassosimone self-assigned this Feb 9, 2021
@bassosimone bassosimone mentioned this issue Feb 9, 2021
Merged
@bassosimone bassosimone added triage Should be triaged through the triage board and removed triage Should be triaged through the triage board labels Feb 10, 2021
@bassosimone bassosimone mentioned this issue Mar 11, 2021
Merged
bassosimone added a commit to ooni/probe-cli that referenced this issue Mar 11, 2021
@bassosimone
refactor+doc(oonimkall): improve docs and simplify code (#253)
78b5bf0 
This diff has been extracted from a larger diff written for ooni/probe#1346
@bassosimone
Copy link
Contributor Author

Requested review to @hellais and/or @lorenzoPrimi here: ooni/probe-cli#223

@bassosimone
Copy link
Contributor Author

I'm going to tentatively move this to the next Sprint.

@bassosimone bassosimone mentioned this issue Mar 18, 2021
Closed
@bassosimone
Copy link
Contributor Author

Created follow-up issue here #1410.

ainghazal pushed a commit to ainghazal/probe-cli that referenced this issue Mar 8, 2022
@bassosimone
refactor+doc(oonimkall): improve docs and simplify code (ooni#253)
c47420f 
This diff has been extracted from a larger diff written for ooni/probe#1346
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bassosimone bassosimone

Labels
priority/high Important issue that needs attention soon
Projects
None yet
Milestone
Sprint 35 - Octariinae
Development

No branches or pull requests
2 participants
@bassosimone @hellais