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

[iOS] Metal-Prerelease: Map Snapshotter fails on simulator #1979

Closed
hactar opened this issue Dec 20, 2023 · 2 comments
Closed

[iOS] Metal-Prerelease: Map Snapshotter fails on simulator #1979

hactar opened this issue Dec 20, 2023 · 2 comments
Labels
bug Something isn't working iOS metal

Comments

@hactar
Copy link
Contributor

hactar commented Dec 20, 2023

Describe the bug
When using the metal prerelease, for example ios-v6.0.0-pre8820165ccafb4f3db109b0b4ae91739d577d7ece you cannot use MLNMapSnapshotter on the iOS simulator, for example an iPhone Pro 15 (17.2) simulator. The code builds and runs fine, but during snapshotting, the app crashes with the following message:

-[MTLTextureDescriptorInternal validateWithDevice:]:1357: failed assertion `Texture Descriptor Validation
MTLTextureDescriptor: Depth, Stencil, DepthStencil textures cannot be allocated with MTLStorageModeShared on this device.

On real devices, for example my iPhone 12 Pro Max running iOS 17.1 the snapshotter works fine.

To Reproduce

  • Build and run the attached project on the simulator.
  • Hit the "Generate Map Snapshot"

Expected behavior
A UIImage of the map should be generated. Instead the app crashes as described above.

Screenshots
On device, this works, the final result being this:

Platform information (please complete the following information):

  • iOS 17.2 Simulator
  • Version ios-v6.0.0-pre8820165ccafb4f3db109b0b4ae91739d577d7ece

Example App
MapLibreSnapshotterExample.zip

@hactar hactar added the bug Something isn't working label Dec 20, 2023
@louwers
Copy link
Collaborator

louwers commented Dec 20, 2023

Thanks for the report! This is expected. The Snapshotter still needs to be implemented for Metal. #1862

@louwers louwers closed this as not planned Won't fix, can't repro, duplicate, stale Dec 20, 2023
@louwers
Copy link
Collaborator

louwers commented Dec 20, 2023

I updated the pre-release overview thread. #1609

Please let us know if you come across anything else!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working iOS metal
Projects
None yet
Development

No branches or pull requests

2 participants