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

This commit adds a basic change-watcher of a loaded file with auto-up… #337

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

susnicek
Copy link

Dear developer,

first of all I must thank you very much for GPXSee map viewer, which is free and is perfect!
Please, take this pull request as an inspiration for implementation of an auto-update system of loaded files. I use this feature for displaying current coordinates from my GPS device. My external script downloads data from GPS device and writes them into a file, which is loaded in GPXSee program. With the introduced auto-update system the position from my GPS device is updated after each file change.
My pull request introduces this feature in its simplest form. The feature is implemented as default and cannot be switched off. It is so because I did not want to modify your GUI and wanted you to decide how to implement the switching on or off the feature.
I hope that you will find the idea interesting and aligned with your future vision of GPXSee map viewer.

Best regards, Susnicek

@tumic0
Copy link
Owner

tumic0 commented Jan 14, 2021

Hi. The idea of auto-updates is nice. I would not see this as the optimal way how to obtain the device's location - there is QtPositioning for that - but it may be quiet usable as a "poor man's live tracking system". However, it requires some more changes in the GUI, the paths rendering logic (the markers) and maybe also in the path info handling to make it a real feature. But I will have a look at it.

@susnicek
Copy link
Author

Yes, I agree, it definitely needs more changes (also e.g. zoom handling on file update).
Thank you very much for having a look at it.

Best regards, Susnicek

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.

2 participants