microg / IchnaeaNlpBackend

Backend for UnifiedNlp that uses Mozilla Location Service for geolocation.
119 stars 28 forks source link

Questions about widespread deployment #32

Closed crankycoder closed 5 years ago

crankycoder commented 6 years ago

Hi!

I wasn't sure if there was a better place to ask general questions - but is there any data on how widely the microG library is being deployed? We're seeing a lot of traffic on Ichnaea and microG is hitting it's API key rate-limit at Mozilla.

ck_location_server_timeboard___datadog

The traffic looks unusual - I think this is probably because of over aggressive querying of the service in the IchnaeaNlpBackend code which spins up a thread everytime onCellChanged() is invoked.

Is there an IRC channel or slack channel where I can ask someone questions?

thanks!

mar-v-in commented 6 years ago

I think this issue is mostly addressed with the v1.4.0 update. I'll leave it open for you to report in a week or so.

crankycoder commented 6 years ago

On our end - we just increased the API key limits for microG so that the user base isn't getting hit with HTTP 400 errors when the limit gets hit a couple hours before midnight UTC.

That should alleviate the immediate problem of users getting no location fix, but I'm hoping the new patch in the IchnaeaNlpBackend that just got landed will be a more permanent fix to reduce traffic load.

crankycoder commented 6 years ago

Where can I track the v1.4.0 release? It's not entirely obvious on microg.org

mar-v-in commented 6 years ago

F-Droid seems to be unable to build the v1.4.0 update, so it's not deployed yet. You can track the status here: https://f-droid.org/wiki/page/org.microg.nlp.backend.ichnaea/lastbuild_20033 The underlying issue (not supporting gradle 4.8) was already fixed 2 weeks ago in their codebase, so all we can do now is wait for them to update their build servers.

rugk commented 5 years ago

So, any news here?

crankycoder commented 5 years ago

The build completed according to the fdroid link above.

A push out to users should have happened. If not - that's an fdroid issue.

On Sun, Jan 13, 2019, 9:49 AM rugk <notifications@github.com wrote:

So, any news here?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/microg/IchnaeaNlpBackend/issues/32#issuecomment-453835795, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAS3QAZt083bhxSxbamovqJPQqTdIYlks5vC0d8gaJpZM4Qk0ei .

rugk commented 5 years ago

So then this issue can be closed as it is fixed, is not it? :smile: