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.

Sign up for GitHub

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

Jump to bottom

android rendering fails: Too many bindings #11281

Closed
mockersf opened this issue Jan 10, 2024 · 0 comments · Fixed by #11307
Closed

android rendering fails: Too many bindings #11281

mockersf opened this issue Jan 10, 2024 · 0 comments · Fixed by #11307
Labels
A-Rendering Drawing game state to the screen C-Bug An unexpected or incorrect behavior O-Android Specific to the Android mobile operating system P-High This is particularly urgent, and deserves immediate attention P-Regression Functionality that used to work but no longer does. Add a test for this!
Milestone
0.13

Comments

@mockersf
Copy link
Member

Bevy version

main since #10057

[Optional] Relevant system information

On the Android emulator

AdapterInfo { name: "SwiftShader Device (LLVM 10.0.0)", vendor: 6880, device: 49374, device_type: VirtualGpu, driver: "?", driver_info: "?", backend: Vulkan }

What you did

cargo apk run --package bevy_mobile_example

What went wrong

Crash with the following error:

wgpu error: Validation Error
Caused by:
    In Device::create_bind_group_layout
      note: label = `mesh_view_layout`
    Too many bindings of type SampledTextures in Stage ShaderStages(FRAGMENT), limit is 16, count was 21
@mockersf mockersf added C-Bug An unexpected or incorrect behavior A-Rendering Drawing game state to the screen O-Android Specific to the Android mobile operating system P-Regression Functionality that used to work but no longer does. Add a test for this! labels Jan 10, 2024
@mockersf mockersf mentioned this issue Jan 10, 2024
Closed
@nicopap nicopap added the P-High This is particularly urgent, and deserves immediate attention label Jan 10, 2024
@mockersf mockersf added this to the 0.13 milestone Jan 10, 2024
@mockersf mockersf mentioned this issue Jan 11, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Jan 12, 2024
@mockersf
Revert "Implement minimal reflection probes. (#10057)" (#11307)
3d99663 
# Objective

- Fix working on macOS, iOS, Android on main 
- Fixes #11281 
- Fixes #11282 
- Fixes #11283 
- Fixes #11299

## Solution

- Revert #10057
Ixentus pushed a commit to Ixentus/bevy that referenced this issue Jan 12, 2024
@mockersf @Ixentus
Revert "Implement minimal reflection probes. (bevyengine#10057)" (bev…
f5324f3 
…yengine#11307)

# Objective

- Fix working on macOS, iOS, Android on main 
- Fixes bevyengine#11281 
- Fixes bevyengine#11282 
- Fixes bevyengine#11283 
- Fixes bevyengine#11299

## Solution

- Revert bevyengine#10057
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Rendering Drawing game state to the screen C-Bug An unexpected or incorrect behavior O-Android Specific to the Android mobile operating system P-High This is particularly urgent, and deserves immediate attention P-Regression Functionality that used to work but no longer does. Add a test for this!
Projects
None yet
Milestone
0.13
Development

Successfully merging a pull request may close this issue.

Revert "Implement minimal reflection probes. (#10057)" mockersf/bevy
2 participants
@mockersf @nicopap