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

Puck CPU Usage 30-60% even with Pulsing Disabled #1978

Closed
samskinner32 opened this issue Jun 9, 2023 · 1 comment
Closed

Puck CPU Usage 30-60% even with Pulsing Disabled #1978

samskinner32 opened this issue Jun 9, 2023 · 1 comment
Labels
bug 🪲 Something is broken!

Comments

@samskinner32
Copy link

Environment

  • Xcode version: 14.3.1
  • iOS version: 16.4
  • Devices affected: Simulator
  • Maps SDK Version: 10.13.1

Observed behavior and steps to reproduce

Our map usage is fairly straightforward, but we've noticed that the puck2D is causing CPU usage in XCode debugger to hover at 30-60% even while idling on the map. If we remove the puck, we sit at 0-5%, as we'd want. I tried disabling the animation via the following code and it had no effect, but maybe I'm not successfully disabling it?

var pulse = Puck2DConfiguration.Pulsing()
pulse.isEnabled = false
map.location.options.puckType = .puck2D(Puck2DConfiguration(pulsing: pulse))

Expected behavior

I would expect the puck to not increase CPU usage so drastically.

@samskinner32 samskinner32 added the bug 🪲 Something is broken! label Jun 9, 2023
@OdNairy
Copy link
Contributor

OdNairy commented Jun 16, 2023

Thanks for the report, @samskinner32. I'm closing this ticket in favor of #1939.
Please, take a look at the provided workaround options: #1939 (comment)

@OdNairy OdNairy closed this as completed Jun 16, 2023
mapbox-github-ci-writer-public-1 bot pushed a commit that referenced this issue Jan 15, 2024
* Make padding optional in `cameraForCoordinateBounds` and `cameraForCoordinates`

* Add changelog entry

* Update API breakage allowlist
mapbox-github-ci-writer-public-1 bot pushed a commit that referenced this issue Jan 17, 2024
* Make padding optional in `cameraForCoordinateBounds` (#1978)

* Make padding optional in `cameraForCoordinateBounds` and `cameraForCoordinates`

* Add changelog entry

* Update API breakage allowlist

* Add missing dependencies to DebugApp (#1983)

* Add missing dependencies to DebugApp

* Make MAPBOXMAPS_PATH project-specific, not target-specific as it's redundant

* Remove redundant artifacts (#1969)

* Update release readme maps version

* Remove xcodebuild log generated by a wrapper

* Update SDK version to 11.1.0

* Update CoreMaps/Common

* Changelog

* Update license

---------

Co-authored-by: Roman Laitarenko <[email protected]>
Co-authored-by: Aleksei Sapitskii <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something is broken!
Projects
None yet
Development

No branches or pull requests

2 participants