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

Determine the correct scheme to backfill points (LocationService) #394

Closed
kenpugsley opened this issue Apr 8, 2020 · 4 comments
Closed
Labels
High Priority High importance / High value

Comments

@kenpugsley
Copy link
Collaborator

#389 introduces a limit on the amount of time we will try to backfill. Filing this issue to come back to that logic and be backfill and make sure we're doing the best we can do.

@kenpugsley kenpugsley added For next release High Priority High importance / High value labels Apr 8, 2020
@JohnPalmer
Copy link

I suggest that if we use backfilling, we mark the locations that are added this way so that analysts can distinguish them from the observed locations. This would mean adding a new key to the json file and thus increasing the size, but I worry that we may be distorting the data otherwise and making it impossible for someone to later fix it. For example, we may later realize that there is a better approach to dealing with gaps (this is something that I work on) that we would like to implement in the app and/or in the Safe Places tool for public health authorities analyzing the data. If we do not have the backfilled locations marked, we (or they) will not be able to do anything to apply the alternative approach to already-gathered data.

@kenpugsley
Copy link
Collaborator Author

Thanks @JohnPalmer - that's a great idea. We're also looking at if the backfill is truly necessary. The main reason we do it is to cover the case where we don't get location updates from the device because it is not moving. It's unclear if this is actually an issue. I think you're comments above might also be helpful in letting us know if this is indeed something we need to do.

@JohnPalmer
Copy link

Thanks @kenpugsley. This also connects to #403, which I just commented on as well. Since this is something I spent a lot of time thinking through a few years ago, I will focus on it more now.

@penrods
Copy link
Contributor

penrods commented Apr 18, 2020

Addressed now. Closing.

@penrods penrods closed this as completed Apr 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
High Priority High importance / High value
Projects
None yet
Development

No branches or pull requests

3 participants