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

Maps unable to set location/ loading takes too much time after switching from Mapbox #5633

Closed
dbemke opened this issue Jun 15, 2023 · 2 comments · Fixed by #5636
Closed

Maps unable to set location/ loading takes too much time after switching from Mapbox #5633

dbemke opened this issue Jun 15, 2023 · 2 comments · Fixed by #5636
Assignees
Milestone

Comments

@dbemke
Copy link

dbemke commented Jun 15, 2023

ODK Collect version

2023.1.1, the store version 2023.1.2

Android version

10, 13

Device used

Redmi 9T, Galaxy A23

Problem description

After switching from Mapbox to any other source of maps the time of searching for location on a map (e.g. in geopoint with map appearance) is too long. I’m not sure what are the factor causing that but the time varies from 30 sec to 5 minutes.

Steps to reproduce the problem

  1. Google maps are set in project settings
  2. Go to All widgets any Geo widget with maps – check if the map zooms to the location (in reasonable time).
  3. Go to project settings and change the Basemap source to Mapbox.
  4. Go to All widgets any Geo widget with maps – check if the map zooms to the location.
  5. Go to project settings and change the Basemap to any other source (Google, OSM, Carto etc.)
  6. Go to All widgets any Geo widget with maps – check if the map zooms to the location.

Expected behavior

The maps should set location in reasonable time

@github-project-automation github-project-automation bot moved this to not ready in ODK Collect Jun 15, 2023
@grzesiek2010 grzesiek2010 self-assigned this Jun 15, 2023
@seadowg seadowg added this to the v2023.2 milestone Jun 15, 2023
@seadowg seadowg moved this from not ready to in progress in ODK Collect Jun 15, 2023
@grzesiek2010
Copy link
Member

According to my thorough investigation, the issue has been around since we moved to Mapbox v10 in v2022.3.0 which means we have released three versions with that bug v2022.3.x, v2022.4.x, v2023.1.x. Taking it into account and the fact that switching between map engines is probably not a very common scenario I think it's not necessary to fix the problem ASAP in this milestone. However, I've spent a lot of time trying to figure out what's going on and playing with different versions/solutions so I'm going to continue working on this issue as a part of v2023.3 as I have some ideas in my head.

@dbemke
Copy link
Author

dbemke commented Jun 26, 2023

Another issue that might be connected with this is missing location in audit logs in forms with background location tracking. After switching from Mapbox to a different source of maps in audit logs there are "location tracking enabled" and "location providers enabled" events but the location values are missing. If I force stop the app and fill the form again the location values are present in audit logs.

@github-project-automation github-project-automation bot moved this from in progress to done in ODK Collect Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: done
Development

Successfully merging a pull request may close this issue.

3 participants