Wrong uphill calculation

osmandapp / OsmAnd-iOS

OsmAnd for iOS

Other

279 stars 90 forks source link

Wrong uphill calculation #3865

Closed dmpr0 closed 3 weeks ago

dmpr0 commented 2 months ago

Description

How to reproduce

Calculate Pedestrian route from 46.37931° N, 7.62674° E, to 46.39629° N, 7.61146° E.
Disable Use elevation data

Actual result

Uphill 440 m

Expected result

Uphill close to 800 m

Environment Both Android and iOS

vshcherb commented 2 months ago

2024-06-24 Pedestrian 7.83 km.gpx.zip

vshcherb commented 1 month ago

Website shows - Uphill/downhill: 1274/381 m

sonora commented 1 month ago

Website shows - Uphill/downhill: 1274/381 m

Just checked our old "trend channel analysis" for this: OsmAnd 4.0.8 reports 1311 m / 419 m.

vshcherb commented 1 month ago

Still it's not "Uphill 440 m" so the bug is not reproduced, probably on iOS

sonora commented 1 month ago

I really meant "OsmAnd 4.0.8", so our "old" mechanism. :wink:

OsmAnd 4.8.0 (Android) gives 1274 m / 381 m, I just checked. And I just notice that by "Website" you mean not just any GPX online analyzer, but probably our own osmand.net/map. So yes, it seems like an iOS bug then... cannot reproduce on Android.

dmpr0 commented 1 month ago

Still it's not "Uphill 440 m" so the bug is not reproduced, probably on iOS

Android	iOS

OsmAnd~ 4.8.0#2821m, released: 2024-07-23

vshcherb commented 4 weeks ago

As we need more testing this task is valid Android / iOS for 4.9

alex-osm commented 3 weeks ago

https://github.com/osmandapp/OsmAnd-iOS/commit/c0fd8ee1575f25fe0296116c043ecf35dc1cf15b