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

Change default curve range for particle angles to [-1, 1] #91421

Closed
wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented May 1, 2024

Change the default curve range for particle angles from [-360, 360] to [-1, 1].

Fixes #91404

@ghost ghost marked this pull request as draft May 1, 2024 18:14
@AThousandShips AThousandShips added this to the 4.3 milestone May 1, 2024
@ghost ghost marked this pull request as ready for review May 1, 2024 18:15
@akien-mga
Copy link
Member

akien-mga commented May 1, 2024

I didn't test, but this might also need to be adjusted for CPUParticles3D and CPUParticles2D in scene/[23]d/cpu_particles_[23]d.cpp which have the same _adjust_curve_range code.

Note: Please make modifications by amending the commit, and force pushing the changes, so that it stays as a single commit (see PR workflow).

@ghost ghost self-requested a review as a code owner May 1, 2024 18:38
@ghost ghost self-requested a review as a code owner May 1, 2024 18:39
@akien-mga
Copy link
Member

Looks good! Could you squash the commits? See PR workflow for instructions.

@ghost ghost closed this by deleting the head repository May 1, 2024
@KoBeWi KoBeWi removed this from the 4.3 milestone May 1, 2024
@KoBeWi KoBeWi added the archived label May 1, 2024
@aXu-AP
Copy link
Contributor

aXu-AP commented May 4, 2024

Was there some problem with this pr? Is it salvageable?

@akien-mga
Copy link
Member

The changes are correct, but the OP seems to have deleted their GitHub account... So yeah, someone else needs to redo it.

@aXu-AP
Copy link
Contributor

aXu-AP commented May 4, 2024

I'm available, I'll do it

This pull request was closed.
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.

ParticleProcessMaterial: Default curve range for particle angles of -360 to 360 is not very useful
4 participants