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

Clustering: Bad framerate on new devices #411

Open
blurpy opened this issue May 28, 2022 · 6 comments
Open

Clustering: Bad framerate on new devices #411

blurpy opened this issue May 28, 2022 · 6 comments
Assignees
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@blurpy
Copy link
Contributor

blurpy commented May 28, 2022

Enabling clustering makes moving around the map really choppy, as in the framerate stutters.
My app has been using clustering for many years, and this issue seems to be happening to newer devices. I noticed when I got an iPhone 13. I have asked some of my users with older devices, and they don't experience this issue.

My app don't really show many markers. Around 500 at the most. If I disable clustering and render all the markers then performance is great (though usability suffers due to the crowded map). If I enable clustering, even zooming out so I only see a couple of clusters and no other markers, it still stutters the same.

The offending code seems to be this:

GMSCoordinateBounds *visibleBounds =
[[GMSCoordinateBounds alloc] initWithRegion:[_mapView.projection visibleRegion]];

If I comment out that line of code and just set shouldShowCluster to always true then it works smooth again. For some reason the performance has an issue with using map projection.

There is a very relevant bug report for Google Maps here: https://issuetracker.google.com/issues/219068007

The app I have in production right now is using Google Maps v2.7.0 and utils v2.0.0, and it has this problem. Updating to newest version of everything does not help.

The issue is either in iOS or Google Maps, so unless you have a good workaround that does not use projection, then this probably have to wait for someone else to fix it.

Environment details

iOS: v15.4.1
Device: iPhone 13
Google Maps: v2.7.0, v6.2.1
Google Maps iOS Utils: v2.0.0, v4.1.0

Steps to reproduce

  1. Follow the steps here to setup the demo app: https://developers.google.com/maps/documentation/ios-sdk/utility/setup#get-demo
  2. Install the demo app on an iPhone 13
  3. Move around the map and notice how the framerate is really bad
@blurpy blurpy added triage me I really want to be triaged. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels May 28, 2022
@yunnnyunnn
Copy link

I have the same issue too.

@wangela
Copy link
Member

wangela commented Jun 14, 2022

Thank you for bringing this to our attention and linking the bug report from the Issue Tracker. We'll provided updates here as we see updates to the SDK issue.

@blurpy
Copy link
Contributor Author

blurpy commented Jun 15, 2022

I have the same issue too.

Remember to vote on the Google Issue Tracker as well, so they know people are affected.

@blurpy
Copy link
Contributor Author

blurpy commented Jul 15, 2022

They closed the issue without any explanation:

Status: Won't Fix (Obsolete)

Do anyone here have any insider details about why it's obsolete? Tried asking about it, but I doubt they are going to answer.

@marfurt
Copy link

marfurt commented Dec 22, 2022

Hi @wangela, is there any feedback about this issue?

We're experiencing the same issue, but no update has been provided here nor in the Google Issue Tracker.

@blurpy Have you found a solution or a workaround in the meantime?

@blurpy
Copy link
Contributor Author

blurpy commented Dec 23, 2022

@marfurt I'm using the workaround mentioned in the description, disabling the map projection code. Can't use this library without, as the frame drop is really bad.

@iHackSubhodip iHackSubhodip added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Nov 9, 2024
@wangela wangela removed the triage me I really want to be triaged. label Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

No branches or pull requests

5 participants