thomaschampagne / elevate

A sport app to "Elevate" your training experience and goals! Track your fitness and progressions over time. Analyse deeper your activities. And more...
https://thomaschampagne.github.io/elevate-docs/
Mozilla Public License 2.0
1.28k stars 176 forks source link

[Web-Ext][Bug] Cadence incorrect for walking activities, at least in activities tab #1204

Open bartvanandel opened 2 weeks ago

bartvanandel commented 2 weeks ago

Describe the bug Activities nicely show cadence for both cycling and walking (running, hiking, ...) activities. For cycling they look fine to me, but for walking, they are off by a factor of 2. I guess 2 steps are counted as 1 RPM, which for walking doesn't make much sense. You'd never ask someone "count 10 paces" and actually mean 20 footsteps. So, maybe it's better to display SPM (steps per minute) for walking type activities.

To Reproduce Steps to reproduce the behavior:

  1. Go to 'Activities' tab
  2. Find a walking activity
  3. Look at the value in the 'Average cadence' column
  4. Conclude that the value is quite low

Expected behavior Value should be twice what is currently displayed (e.g. 110 instead of 55), and use "SPM" instead of "RPM".

Environment