Skip to content

PoC: running experiments bypassing internal/engine #1075

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
Draft
bassosimone wants to merge 18 commits into
base: master
Choose a base branch
from
Draft

PoC: running experiments bypassing internal/engine #1075

bassosimone wants to merge 18 commits into from

Conversation

@bassosimone
Copy link
Contributor

@bassosimone bassosimone commented Feb 6, 2023

This PoC investigates whether it would be possible to run experiments directly without using the internal/engine abstraction as the middle man.

The PoC is in the context of ooni/ooni.org#1295

Checklist

  • I have read the contribution guidelines
  • reference issue for this pull request:
  • if you changed anything related to how experiments work and you need to reflect these changes in the ooni/spec repository, please link to the related ooni/spec pull request:
  • if you changed code inside an experiment, make sure you bump its version number

Description

Please, insert here a more detailed description.

@bassosimone
poc: running experiments bypassing internal/engine
5ab826e 
This PoC investigates whether it would be possible to run
experiments directly without using the internal/engine
abstraction as the middle man.

The PoC is in the context of ooni/ooni.org#1295
@bassosimone bassosimone changed the title poc: running experiments bypassing internal/engine PoC: running experiments bypassing internal/engine Feb 6, 2023
@bassosimone
Merge branch 'master' into dismantle
b09cc46
@bassosimone
refactor: create the sessionhttpclient package
f25f14a
@bassosimone
doc(model): document location.go data structure
258b8e0
@bassosimone
refactor: facilitate making geolocate.Results a Location
f1d7aed
@bassosimone
fix(model): make locationProvider model complete
c1aa819
@bassosimone
refactor: rewriten measurement constructor to use locationProvider
2a57df7
@bassosimone
refactor: avoid & and * dance
f2572e6
@bassosimone
refactor: move Measurement constructor inside model
4f1a217
@bassosimone
refactor: move descriptor creation into ooapi
e44839b
@bassosimone
refactor: document and improve backendclient implementation
888a94e
@bassosimone
refactor: make backendclient a toplevel package
5204b13
@bassosimone
it works(TM)
1585ec9
@bassosimone
better code
1afc71f
@bassosimone
start modifying ooniprobe
364bae9
@bassosimone
poc: new code for running experiments for ooniprobe
8402b7b
@bassosimone
poc: show how to improve ooniprobe with ~nice code
c06b885
@bassosimone
more experimental code
b2b1f06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hellais hellais Awaiting requested review from hellais hellais will be requested when the pull request is marked ready for review hellais is a code owner

@DecFox DecFox Awaiting requested review from DecFox DecFox will be requested when the pull request is marked ready for review DecFox is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bassosimone