Skip to content

Conversation

@caglarpir
Copy link
Contributor

Summary:
In timelapse videos video time and gps track time have different scales.
Use the gps track timescale when calculating average speed of the gps track.

@caglarpir caglarpir requested a review from ptpt January 14, 2026 15:39
@meta-cla meta-cla bot added the cla signed label Jan 14, 2026
@caglarpir caglarpir force-pushed the gps-timestamp-speed branch 2 times, most recently from 287da89 to 12bebaf Compare January 14, 2026 15:55
Summary:
In timelapse videos video time and gps track time have different scales.
Use the gps track timescale when calculating average speed of the gps track.
@caglarpir caglarpir force-pushed the gps-timestamp-speed branch from 12bebaf to de3098b Compare January 15, 2026 09:32
Copy link
Member

@ptpt ptpt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove the dependencies on telemetry as a follow up PR? Because geo.py is used in multiple places and it'd be better to stay generic. One idea could be moving specific implementations of avg_speed and _interpolate_segment to each Point class

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants