Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tracker optimisation focus #5967

Merged
merged 9 commits into from
Feb 6, 2025
Merged

Tracker optimisation focus #5967

merged 9 commits into from
Feb 6, 2025

Conversation

nirvn
Copy link
Member

@nirvn nirvn commented Jan 30, 2025

This PR implements a number of strategies to optimize tracking playback when returning from a long period of time when QField was set in the background or the phone was locked.

For line and polygon layers, this makes replaying hours of captured positions near instantaneous.

For point layers, the speed is significantly improved, yet as every position will lead to a completely new feature being added (inc. unique value checks, default value expression calculation, etc), the time required to playback the position can take longer. To that end, we now have a busy indicator overlay to tell the user an ongoing operation is happening.

Overall, it makes using background tracking for long period of time much, much more efficient.

@qfield-fairy
Copy link
Collaborator

qfield-fairy commented Jan 30, 2025

@nirvn nirvn force-pushed the tracker_optimisation branch 2 times, most recently from a12dff9 to 4371639 Compare February 1, 2025 10:33
@nirvn
Copy link
Member Author

nirvn commented Feb 4, 2025

@m-kuhn , did you give this a try? Can we merge?

@nirvn nirvn force-pushed the tracker_optimisation branch from 4371639 to d8c652c Compare February 4, 2025 03:58
@m-kuhn
Copy link
Member

m-kuhn commented Feb 4, 2025

@m-kuhn , did you give this a try? Can we merge?

Not yet, it says "WIP" in the description :-)

@nirvn
Copy link
Member Author

nirvn commented Feb 4, 2025

@m-kuhn , oups! :) fixed the description, and about to fix your review. Thanks.

@nirvn nirvn force-pushed the tracker_optimisation branch from d8c652c to a4471cc Compare February 4, 2025 08:30
src/core/tracker.cpp Outdated Show resolved Hide resolved
@nirvn nirvn force-pushed the tracker_optimisation branch from a4471cc to bccb67e Compare February 4, 2025 08:35
@nirvn nirvn merged commit e91c10f into master Feb 6, 2025
25 checks passed
@nirvn nirvn deleted the tracker_optimisation branch February 6, 2025 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants