Fix missing particles on android #270
Conversation
Fix particles not rendering on android due to GPU particles not being supported. Implements a compatibility class to help convert GPU particles to CPU particles at runtime - might need some work.
GTcreyon
added
bug
GTcreyon
requested review from
AidenBeresford,
Koopa1018,
jaschutte and
Kuma-Boo
|
Should be all fixed now. |
|
@Koopa1018 Could this get re-reviewed? |
|
D'oh, slipped my mind for a while! Looks like I got the entire build toolchain working before I forgot. Keystore usernames are still kicking my butt, but I've managed to build and test. Turns out, this bug does not occur for me, at least not in debug builds. Particles show properly whether I'm on master or the fix branch. Can you confirm whether debug vs. release has any effect for you? Because I might not actually be able to test this if not! |
To be clear... you are testing on Android, right? It's an Android-specific bug. It also might be specific to my phone hardware. I will make sure to test with debug on master. |
|
Yeah, testing on a real Android phone (Motorola Moto G Play, not sure of the more specific details). Runs crazy slow (I blame debug builds), but it shows particles fine. |
There was a problem hiding this comment.
You know what, I'm inclined to just approve this, on the basis that it doesn't seem to have broken anything on my end.
That said: @jaschutte @AidenBeresford @Kuma-Boo extra tests always welcome.
|
Hm. I think I might just close this for now. I'm not very fond of the solution I came up with. It's hacky and weird, and hacky-weird stuff tends to cause unexpected compatibility issues, which is exactly what this PR was trying to avoid. Besides, the benefit from using GPU particles seems negligible. Moreover, platforms that can handle GPU particles probably don't actually need them, so using this hacky solution at the risk of breaking functionality just so the rich get richer seems unreasonable. Godot has also progressed in this time. The issue may have resolved itself, or affected devices may no longer be supported at all. The issue no longer occurs on my device, for one. Closing for now. If anyone can replicate this issue, let me know. |
Description of changes
Fix particles not rendering on android due to GPU particles not being supported. Implements a compatibility class to help convert GPU particles to CPU particles at runtime - might need some work.
Issue(s)
Closes #269