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

[Fancier Weather] Digging and snow particles only rotated on Y axis #20

Open
Meldexun opened this issue Feb 16, 2023 · 1 comment · May be fixed by #8
Open

[Fancier Weather] Digging and snow particles only rotated on Y axis #20

Meldexun opened this issue Feb 16, 2023 · 1 comment · May be fixed by #8
Assignees
Labels
Bug Something isn't working as intended
Milestone

Comments

@Meldexun
Copy link

Meldexun commented Feb 16, 2023

Describe the bug

Digging and snow particles are only rotated around Y axis. In the original FBP digging and snow particles were rotated around the X, Y and Z axis. And depending on the config options they aligned themselves when hitting the ground.

This happens because the code that aligns the digging and snow particles now ignores whether they are on the ground or not. And also because the restOnFloor option was removed.

These are the lines that align the particles:
https://github.com/Red-Studio-Ragnarok/Fancier-Block-Particles/blob/0.8-Refractor-%26-Cleanup/src/main/java/io/redstudioragnarok/FBP/particle/FBPParticleDigging.java#L303-L304
and
https://github.com/Red-Studio-Ragnarok/Fancier-Block-Particles/blob/0.8-Refractor-%26-Cleanup/src/main/java/io/redstudioragnarok/FBP/particle/FBPParticleSnow.java#L144-L145

EDIT: Also in the FastCubeUploader class the rotation is applied in the wrong order. The quaternion should be initialised with the Y axis rotation and the X axis rotation should be done last.
https://github.com/Red-Studio-Ragnarok/Fancier-Block-Particles/blob/0.8-Refractor-%26-Cleanup/src/main/java/io/redstudioragnarok/FBP/renderer/FastCubeUploader.java#L24-L26

To Reproduce

  1. Break blocks to spawn digging particles/go to snowy biome and set weather to rain to spawn snow particles
  2. Notice that digging and snow particles are only rotated around the Y axis

Expected behavior

I expected that the digging and snow particles are rotated around the X, Y and Z axis while in air and (depending on the config) also while on the ground.

Screenshots or/and videos

This is the current state. You can see that the snow particles in air and on the ground are only rotated around the Y axis.

Minecraft.1.12.2.2023-02-15.21-16-10.mp4

Here I added a check if the particle is on ground to the code that aligns the particle if (onGround) { ... }.

Now the snow particles rotate properly again while in the air and get aligned when on the ground. This is how it used to be in the original FBP when you had restOnFloor=true.

Minecraft.1.12.2.2023-02-15.21-16-37.mp4

Here I just removed the code that aligns the particle altogether.

Now the snow particles rotate again while in the air and and while on the ground. This is how it used to be in the original FBP when you had restOnFloor=false.

Minecraft.1.12.2.2023-02-15.21-17-02.mp4

Versions

Specification:

  • OS: Windows 10
  • CPU: Ryzen 7 5600
  • GPU: RTX 2070
  • RAM Allocated: 1GB

All of these info below are not necessary but may become necessary depending on the issue

  • CPU Speed: 4.2GHz
  • CPU Core: 6
  • CPU Threads: 12
  • RAM Size: 32GB
  • RAM Speed: 3200MHz
  • Storage Type: SSD
  • GPU VRAM: 8GB
@Desoroxxx Desoroxxx added Bug Something isn't working as intended Need Investigation labels Feb 16, 2023
@Desoroxxx Desoroxxx added this to the 0.8 milestone Feb 16, 2023
Desoroxxx added a commit that referenced this issue Feb 16, 2023
@Desoroxxx Desoroxxx linked a pull request Feb 16, 2023 that will close this issue
@Desoroxxx
Copy link
Member

Fixed in dev build 13

@Desoroxxx Desoroxxx changed the title Digging and snow particles only rotated on Y axis [Fancier Weather] Digging and snow particles only rotated on Y axis Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working as intended
Development

Successfully merging a pull request may close this issue.

2 participants