-
Notifications
You must be signed in to change notification settings - Fork 0
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
Latest Steps Not Added To DB/Graph? #145
Comments
Ok as shown here, his steps are indeed being tracked but not being added to the graph. |
The steps don't get added to the graph. I thought previously it was because the steps are so little so they just appear as 0 but I guess not |
Hmm that is interesting, did you test it 22 minutes ago (from time of my writing)? Because I just got the same PlatformException again in Sentry. |
Was this after using the testing page to schedule the checkup? |
No it was by rescheduling in settings |
Okay can you work on this, it's the priority for now. I can't reproduce it. |
Doesn't add the steps to the graph even when I use the Wellbeing Check Screen button in Testing |
Review #151, it may fix it. |
I did, that's what I was testing |
Using WellbeingCheck Notification button in testing, I get this. W/System (11762): A resource failed to call release. Later, as I move the phone: W/FlutterJNI(11762): Tried to send a platform message to Flutter, but FlutterJNI was detached from native C++. Could not send. Channel: step_count. Response ID: 0 |
The platform detached message should be fine to ignore for now. When does that Platform exception occur, after you press the done button? Does everything look normal from the user's perspective (e.g. the steps load)? |
I think yes. Yeah everything looks normal. What do you mean by steps load? The step count increases as I move the phone and after checkup, goes back to 0 like it should |
No, the exception appears when I click on the wellbeing page after checkup |
That's interesting. You may want to print out the list of wellbeing items like so in wellbeing_graph.dart line 215:
to determine if it's a problem with the graph library or our adding to the DB. Anyway I shall leave it to your judgement for now, but let me know if help is needed. |
It prints out the right amount of steps, so it's being added to the DB correctly. So it's an issue with the graph library |
When I get rid of ..setAttribute(charts.measureAxisIdKey, 'secondaryMeasureAxisId'); on line 145 of wellbeing_graph.dart, it fixes it - steps are displayed |
Hmm that's a little suspicious. Removing that would remove the green annotation on the graph, and scale down the graph. |
But the same amount of steps show up on the iPhone graph |
Oh so this problem only occurs on the Android 10 device? |
Yeah, think so. Gonna double check by running it on the iPhone with the new changes one sec Edit: never mind, it doesn't show up on iPhone either |
I'll carry around the android phone and try again later today when the steps are higher? Or i could just insert a wb item |
This comment has been minimized.
This comment has been minimized.
@saachipahwa Yes try generating a wellbeing item(s) first from testing page and also try performing a checkup after actually walking. |
I increased the intro screen checkup steps to 1000 and it shows up so not a graph library issue?
No, every time I do a checkup it has reset |
Generating from testing page works fine! steps show up on the graph Yep I'll do some walking and see |
I'm starting to think there wasn't an issue to begin with, although we should still prob fix that PlatformException. |
His steps should've showed up on his graph, though. Guess we can ask him to show us what he did tomorrow |
I walked a lot today, did a checkup, and the steps showed up on the graph |
One explanation for this is if he performed a couple checkups (e.g. using testing page) at the same time, so his steps would have been zero. Especially if he just installed the app. |
He has agreed with my explanation. Closing for now. |
https://trello.com/c/i2mA7CAL/97-it-does-not-appear-to-be-registering-my-movement
This seems related, from Sentry:
The text was updated successfully, but these errors were encountered: