-
-
Notifications
You must be signed in to change notification settings - Fork 21.6k
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
GPUParticles2D Turbulence parameters are affected by the rectangular (box) emission shape and particle flags #74541
Comments
Might be fixed by #64606 |
I've just tested this in v4.1.dev3.official [a67d37f], and the issue still persists. Moreover, the new implementation doesn't seem very "turbulent" at all - I'm not sure why this is, but playing with the turbulence settings, I'm not able to get anywhere near the same behaviour as before. Particles just seem to wiggle around a little bit now, but the visual characteristics are completely different. Please let me know if there's something that I'm missing with the new implementation, and I can re-test. Edit: I've filed a new bug report for the unexpected turbulence changes. See #77491 for details |
It seems to have something to do with the distribution of the results of I'd like to better figure out what exactly is causing the issue to begin with, though. |
Hey @KdotJPG , did you figure it out in the end? Otherwise we can just add an extra random call and call it a day. |
Alright, I've tested it with your PR and noticed the following things:
|
Does it happens also if you turn off interpolation? |
Disabling interpolation seemed to fix a lot of the system-wide jitter (the Y-position issue I described above). I'm assuming this behaviour is not intended. Let me know if you want me to open a new bug report for it. The turbulence jitter that occurs when |
Interpolation issues are already being tracked in #51318. |
Thank you @Novark ! I'll investigate the issue with max turbulence and try to address it with my PR. If I don't figure it out I'll address it in a subsequent PR. |
Godot version
v4.0.stable.official [92bee43]
System information
Windows 11 [Vulkan]
Issue description
The turbulence influence min/max properties seem to apply across the emitter shape in a linearly increasing fashion instead of randomly on a per-particle basis as per the documentation:
IMPORTANT: This issue is resolved when the "Disable Z" Particle Flags checkbox is un-checked. This checkbox is enabled by default when the GPUParticles2D node is enabled.
P.S. the documentation for turbulence_influence_min and turbulence_influence max are incorrectly swapped.
Steps to reproduce
To reproduce this is fairly simple:
The expected behaviour would have the turbulence influence take a random value between [min, max] and apply a random value to each particle along the emission shape.
Un-check the "Disable Z" Particle Flag to see the intended behaviour. I'm unsure why this checkbox is affecting the way turbulence influence is distributed across the emission shape, but it may have something to do with the way
VELOCITY
is being calculated in the shader. Digging into the code a bit, it seems that there's some pretty large discrepancies with how the velocity spread is calculated forPARTICLE_FLAG_DISABLE_Z
enabled vs. disabled. I'm not familiar enough with the code to say for certain where this might be occurring, however.Minimal reproduction project
No sample project provided, as the reproduction is quite simple to replicate.
The text was updated successfully, but these errors were encountered: