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

Fix INSTANCE_CUSTOM.w not being assigned correctly in CPUParticles 2D and 3D #90095

Merged

Conversation

QbieShay
Copy link
Contributor

@QbieShay QbieShay commented Apr 1, 2024

This PR fixes an issue with CPUParticles not assigning and using INSTANCE_CUSTOM.w in the same way as GPUParticles, causing any shader that correctly uses lifetime (INSTANCE_CUSTOM.y/INSTANCE_CUSTOM.w) to misbehave when assigned to CPUParticles from GPUParticles.

There's no issue coupled to this because it came out of a troubleshooting conversation on discord that turned into a bug report.

CPU_Particles_Instance_Custom_MRP.zip

@QbieShay QbieShay requested review from a team as code owners April 1, 2024 12:04
@KoBeWi KoBeWi added this to the 4.3 milestone Apr 1, 2024
@AThousandShips AThousandShips changed the title Fixed INSTANCE_CUSTOM.w not being assigned correctly in CPUParticles 2D and 3D Fix INSTANCE_CUSTOM.w not being assigned correctly in CPUParticles 2D and 3D Apr 1, 2024
Copy link
Member

@clayjohn clayjohn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@akien-mga akien-mga changed the title Fix INSTANCE_CUSTOM.w not being assigned correctly in CPUParticles 2D and 3D Fix INSTANCE_CUSTOM.w not being assigned correctly in CPUParticles 2D and 3D Apr 4, 2024
@akien-mga akien-mga merged commit 7c68926 into godotengine:master Apr 4, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants