jonasoreland / runnerup

A open source run tracker
GNU General Public License v3.0
752 stars 275 forks source link

Incorrect speed calc #852

Closed GormCh closed 4 years ago

GormCh commented 4 years ago

The total average time isn't right. Gorm Screenshot_20191123-094911

gerhardol commented 4 years ago

The average pace is calculated from time and distance and is calculated correctly. Maybe autopause (there has been corrections in those calculations in 2.0).

What version are you using? Can you share the tcx?

GormCh commented 4 years ago

Hallo. Version 2.0.2.1

Gorm

Den 23 november 2019 20:39:04 skrev Gerhard Olsson notifications@github.com:

The average pace is calculated from time and distance and is calculated correctly. Maybe autopause (there has been corrections in those calculations in 2.0).What version are you using? Can you share the tcx?— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

GormCh commented 4 years ago

Hi. Should have added that updates to my Runkeeper account doesn't work. It say it's updating but nothing is seen on Runkeeper. It's not a big thing but it did work before.

Gorm

Den 23 november 2019 20:39:04 skrev Gerhard Olsson notifications@github.com:

The average pace is calculated from time and distance and is calculated correctly. Maybe autopause (there has been corrections in those calculations in 2.0).What version are you using? Can you share the tcx?— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

gerhardol commented 4 years ago

Runkeeper works for me (also remove/add the account) but expect it to stop working as rk has removed official support of the api.

The tcx is required to investigate.

GormCh commented 4 years ago

RunnerUp_0098.zip

GormCh commented 4 years ago

Hi! I replied via email on 23rd and attached the file. Tried to to it directly here on Github and got a message that the file type is not accepted. I sent a zip instead.

Gorm

gerhardol commented 4 years ago

Created a PR You can fix the total time by Edit->Recalculate too

GormCh commented 4 years ago

Yeah, that worked. Issue closed. Gorm

gerhardol commented 4 years ago

The change is merged to master