-
Notifications
You must be signed in to change notification settings - Fork 34
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
NO TRIPS RECORDED! updated cistup-ec new mode and purpose changes made out of master. #430
Comments
It is very unlikely that the new update to cistup-ec broke the trip recording, since it was a UI only change that did not change the native plugins in any way. |
Here's what I see from the logs that were sent to me. It looks like this was a new install and not an existing app update
The FSM gets initialized properly and is in
However, we are unable to contact the server during the onboarding. This is the first call to
We continue getting 503 errors.
Including when we try to access the diary.
This pattern continues for a while, but then the connection issue is resolved.
apparently while trying to retrieve data for Jul 1st.
and there is no data on the server. That doesn't seem surprising given that the app was installed fresh on 6th July. Is there data on the server from a prior installation?
Not sure if this is being reported as "no trips recorded", some additional information on which days you are checking and which trips you expect to see would be helpful. |
I couldn't see the update in my app. So I uninstalled it and install it again to test.
Not sure whether it is a connection to server issue. |
After the installation 6th July. I did travel many times. |
I don't get "emTripLog has started tracking" prompt when I travel. No trips in the diary. |
In terms of trips being tracked in this installation, we got a tracking error fairly soon after the installation.
which appears to be because location services was turned off (https://github.com/e-mission/e-mission-data-collection/blob/9ffb867e3af7e4c36dc9d9f0f094b7d0510a9287/src/android/location/GeofenceExitIntentService.java#L77)
During the next periodic check, we do re-initialize the FSM, but that is after noon, so the morning trip is skipped.
this happens again at
and tracking is only turned on at
Are you sure your location services are turned on, and you have WiFi and data turned on? |
At this point, this does not appear to be related to the UX update, but rather, to the tracking error that is periodically generated on your phone. |
And checking the more recent logs that you sent, we got a tracking error
and then there are no logs until the 8th when we are launched again for the periodic sync which
I am not sure why you consistently get an erroneous geofence exit, if you are not turning location services or WiFi on/off, but that is the root cause of your problem. |
Okay I again travelled with the location ON and made sure that no Wifi connects to it. I can see the trips and mode and purpose prompt. |
Great!
Just to clarify, you should be connected to WiFi. The geofence works best when connected to both WiFi and cellular data. Also, couple of follow ups from the testing to improve long-term robustness...
Thanks! |
Once you answer the followups, I will close this issue |
Yup I saw 6 tracking_error altogether. |
Also received logs from an iOS phone reporting a similar issue. On investigation, this also turned out to be a similar root cause - the entire app was reinstalled
and there was apparently an issue while creating the geofence.
Since the geofence creation failed, there was no further tracking. Judging from the documentation, https://developer.apple.com/documentation/corelocation/clerror?language=objc this corresponds
I suspect that the user has provided only "when in use" permissions, not "always". |
It is not clear why users are uninstalling and reinstalling the native app for a UI-only update. |
The reason why is we can't see the update for the cistup-ec channel. We didn't go to the updates tab. Whenever I make any new changes to Logtrip I get a UI update prompt but I clearly understand why its not with the emTripLog as we have different channels we need to update from button. I'll make sure other user while updating use update button. |
Checking on that. |
You should also get a prompt to apply the UI-only update when you launch the app. This has clearly worked before (see #359). I'm a bit surprised that it is not working now. Sometimes, if there are race conditions, the prompt may not display on the first launch after the update, but it should show up in the first couple of launches. |
Closing this issue if there are no additional concerns. |
Something flaky in the new update in cistup-ec. Trips not being recorded.
The text was updated successfully, but these errors were encountered: