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

[3.2.4 beta 4] new pixel snap jitters with odd sized sprites #44686

Closed
PapaCheese opened this issue Dec 25, 2020 · 4 comments
Closed

[3.2.4 beta 4] new pixel snap jitters with odd sized sprites #44686

PapaCheese opened this issue Dec 25, 2020 · 4 comments

Comments

@PapaCheese
Copy link

PapaCheese commented Dec 25, 2020

Godot version:
3.2.4 beta 4

OS/device including version:
windows 10, geforce 1080

Issue description:
the new pixel snap jitters with odd numbered sized sprites (ex. 31x31) with a smooth camera

3.2.3 stable:

Desktop.2020.12.25.-.17.17.14.06.mp4

3.2.4 beta 4 (only 64x64 and 32x32 works properly)

Desktop.2020.12.25.-.17.16.50.05.mp4

Steps to reproduce:
(3.2.4 beta 4) new project, enable pixel snap, stretch mode to viewport, add pixel art sprite with odd size (filter off), add smooth camera and move it in game.

side notes:
i added pixel perfect script to the main scene.
the new 'transform snap' does not solve my issue.

Minimal reproduction project:
pixel snap jitter Minimal reproduction project.zip

@lawnjelly
Copy link
Member

lawnjelly commented Dec 25, 2020

I just tested and it seems to be the extra changes in #43554 (not the m6502 change).

Following on from reduz PR #43194, some more of the snapping is now controlled by the transform snapping project setting, rather than the pixel snapping project setting. I'm not quite sure on the reasoning for this (as this is just a straight backport of his, aside from the canvas (camera) snapping). Actually it does make some logical sense as the pixel_snap is primarily vertex snapping in the vertex shader, whereas the component causing this issue is a transform snap similar-ish to the new transform snap feature.

We could consider keeping control of these with the pixel snap option in 3.2, to keep compatibility. Or have another option to control this snapping, but it would be better to avoid too many settings to prevent things becoming overcomplicated. Hopefully I can find out from reduz the reasoning for moving these settings and the best compromise.

BTW, you say 'the new 'transform snap' does not solve my issue', what other problems were you having with this switched on? It does solve the relative jiggle for me (but I can understand if it causes another problem).

@PapaCheese
Copy link
Author

with 'transform snap' on, walking with my character at a certain speed makes its head jitter (not visible in video)

@lawnjelly
Copy link
Member

lawnjelly commented Dec 25, 2020

with 'transform snap' on, walking with my character at a certain speed makes its head jitter (not visible in video)

Ah this may be because of the camera not being snapped. I will look again at this.

The problem is that currently snapping the camera in reduz method causes another bug to exhibit, which results in a subtle jitter. This can be prevented by doing the camera snapping manually. But this may be too much for many users to understand. Maybe just forcing the camera snap will get someone interested to fix the other bug.

See this post for more info: #43554 (comment)

@akien-mga
Copy link
Member

Fixed by #44690.

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

No branches or pull requests

4 participants