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

Implement faster measurement resubmission #302

Merged
merged 3 commits into from Aug 27, 2019
Merged

Conversation

bassosimone
Copy link
Member

@bassosimone bassosimone commented Jul 25, 2019

Part of ooni/probe#861.

Uses measurement-kit/mkcollector#25 and measurement-kit/mkall-ios#25 to make the act of resubmitting really faster.

This is bundled with https://github.com/measurement-kit/mkall-ios/releases/tag/v0.5.5 that contains also reliability fixes, including a fix to make sure we really deallocate ObjectiveC objects context.

Part of ooni/probe#861.

Uses measurement-kit/mkcollector#25 and
measurement-kit/mkall-ios#25 to make the
act of resubmitting really faster.

This is bundled with
https://github.com/measurement-kit/mkall-ios/releases/tag/v0.5.5
that contains also reliability fixes, including a fix to make
sure we really deallocate ObjectiveC objects context.
@bassosimone bassosimone added this to the OONI Probe 2.1 milestone Jul 25, 2019
@bassosimone bassosimone self-assigned this Jul 25, 2019
@bassosimone
Copy link
Member Author

@lorenzoPrimi please take a look

Copy link
Contributor

@lorenzoPrimi lorenzoPrimi left a comment

Choose a reason for hiding this comment

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

The PR is approved but I need to test it on device first.

@bassosimone
Copy link
Member Author

@lorenzoPrimi there is a branch and PR of the iOS app that you can use for this purpose

@bassosimone
Copy link
Member Author

@lorenzoPrimi updated to use mkall-ios v0.6.0 (and MK v0.10.5). Please, take a look!

@bassosimone
Copy link
Member Author

As discussed with @lorenzoPrimi, I'll leave to merge this PR when it's most convenient for him!

@lorenzoPrimi lorenzoPrimi merged commit 64ebeba into master Aug 27, 2019
@lorenzoPrimi lorenzoPrimi deleted the feature/mkreporter branch August 27, 2019 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants