#2155 Pace and heartrate min/max are inverted in activity summary

Open
opened 2 weeks ago by 8e2ff470e539 · 1 comments

Before reporting a bug, please confirm the following:

I got Gadgetbridge from:

If you got it from Google Play, please note that version is unofficial and not supported here; it's also often quite outdated. Please switch to one of the above versions if you can.

Your issue is:

Best explained with a screenshot:

screenshot-gadgetbridge

This seems wrong as I'd expect:

  • 'maxHR' to be greater than 'Heartrate', and
  • 'Pace' to be correlated to speed, so a slow pace should cover less distance than a fast pace in a given unit of time.

The code for this looks to depend on a load of unknowns in the HuamiActivitySummaryParser, so I suspect this could be device dependent if it works on the original Bip device.

https://codeberg.org/Freeyourgadget/Gadgetbridge/src/branch/master/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/huami/HuamiActivitySummaryParser.java#L137

Can anyone confirm?
Happy to raise a merge request with a fix if so.

Your wearable device is:

Amazfit Bip S 2.1.1.60

Your android version is:

8.1.0

Your Gadgetbridge version is:

0.51.0

#### Before reporting a bug, please confirm the following: - [x] I have read the [wiki](https://codeberg.org/Freeyourgadget/Gadgetbridge/wiki), and I didn't find a solution to my problem / an answer to my question. - [x] I have searched the [issues](https://codeberg.org/Freeyourgadget/Gadgetbridge/issues), and I didn't find a solution to my problem / an answer to my question. - [x] If you upload an image or other content, please make sure you have read and understood the [Codeberg Terms of Use](https://codeberg.org/codeberg/org/src/branch/master/TermsOfUse.md) ### I got Gadgetbridge from: * [x] F-Droid * [ ] I built it myself from source code (specify tag / commit) If you got it from Google Play, please note [that version](https://github.com/TaaviE/Gadgetbridge) is unofficial and not supported here; it's also often quite outdated. Please switch to one of the above versions if you can. #### Your issue is: Best explained with a screenshot: <a href="https://ibb.co/zfJqQSP"><img src="https://i.ibb.co/Hgd5BrG/screenshot-gadgetbridge.png" alt="screenshot-gadgetbridge" border="0"></a> This seems wrong as I'd expect: - 'maxHR' to be greater than 'Heartrate', and - 'Pace' to be correlated to speed, so a slow pace should cover less distance than a fast pace in a given unit of time. The code for this looks to depend on a load of unknowns in the `HuamiActivitySummaryParser`, so I suspect this could be device dependent if it works on the original Bip device. https://codeberg.org/Freeyourgadget/Gadgetbridge/src/branch/master/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/huami/HuamiActivitySummaryParser.java#L137 Can anyone confirm? Happy to raise a merge request with a fix if so. #### Your wearable device is: Amazfit Bip S 2.1.1.60 #### Your android version is: 8.1.0 #### Your Gadgetbridge version is: 0.51.0
vanous commented 1 week ago
Poster
Collaborator

Thank you @8e2ff470e539 for the report, much appreciated. These are busy days, we will follow up as soon as possible.

Thank you @8e2ff470e539 for the report, much appreciated. These are busy days, we will follow up as soon as possible.
vanous added the
device amazfit bip
label 1 week ago
vanous added the
bug
label 1 week ago
vanous added the
activity post processing
label 1 week ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.