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

Different data shown on map for linestring vs geojson #434

Open
genebean opened this issue Nov 27, 2023 · 6 comments
Open

Different data shown on map for linestring vs geojson #434

genebean opened this issue Nov 27, 2023 · 6 comments
Labels

Comments

@genebean
Copy link

These two URLs show different data for some reason:

- /map/index.html?from=2023-11-19T00:43:33&to=2023-11-27T15:43:33&format=geojson&user=gene&device=foo
+ /map/index.html?from=2023-11-19T00:43:33&to=2023-11-27T15:43:33&format=linestring&user=gene&device=foo

For context, I have backfilled some data via http, but created_at is set to the time I ran a script to backfill the data and tst is set to when the data was actually acquired. The backfilled data is what is missing from the linestring version.

$ apt info ot-recorder
Package: ot-recorder
Version: 0.8.6-0-deb10.1
Priority: extra
Section: default
Maintainer: [email protected]
Installed-Size: 1,968 kB
Depends: libcurl3-gnutls, libmosquitto1, liblua5.2-0, libconfig9, libsodium23, liblmdb0
Homepage: http://owntracks.org
License: https://github.com/owntracks/recorder/blob/master/LICENSE
Vendor: OwnTracks.org
Download-Size: 687 kB
APT-Manual-Installed: yes
APT-Sources: http://repo.owntracks.org/debian buster/main amd64 Packages
Description: A lightweight back-end for consuming OwnTracks data from an MQTT broker
@jpmens
Copy link
Member

jpmens commented Nov 27, 2023 via email

@genebean
Copy link
Author

Sure thing. The email address above seems to have been redacted though

@jpmens
Copy link
Member

jpmens commented Nov 27, 2023

Ah, I responded via email, that's why. The address is [email protected].

@genebean
Copy link
Author

Sent just now.

@jpmens
Copy link
Member

jpmens commented Nov 29, 2023

Files received.

@ckrey spent some time looking at the data, and he's found the reason: the listsort() routine we use in Recorder when producing linestring seems to be broken: the sort we need to do to get timestamps (tst) in order to produce a continuous "line" doesn't work properly. This has no effect on the geojson format which just draws the drops on the map.

I've no ETA whatsoever for this fix unfortunately.

The only consolation I have at the moment is that our Frontend doesn't suffer from this bug, and it sports a much nicer interface, so you might wish to give that a spin.

@jpmens jpmens added the bug label Nov 29, 2023
@genebean
Copy link
Author

Awesome, thanks for the update! I'll try Frontend out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants