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

Add Nearest Mipmap Anisotropic filter option to decals and projectors #51531

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Aug 11, 2021

This is consistent with the BaseMaterial3D filtering options. It can be used for high-quality pixel art textures that remain sharp when viewed at oblique angles, but prevents them from becoming grainy thanks to mipmaps.

The options were also reordered to be consistent with the sampling options available in BaseMaterial3D.

Preview

Click to view at full size.

Nearest Mipmap

Nearest Mipmap

Nearest Mipmap Anisotropic (16×)

Nearest Mipmap Anisotropic

@Calinou Calinou force-pushed the decal-projector-add-nearest-mipmap-aniso branch from 8595122 to eeee42b Compare July 20, 2022 20:49
@akien-mga
Copy link
Member

There seems to be style issues.

@Calinou Calinou force-pushed the decal-projector-add-nearest-mipmap-aniso branch from eeee42b to 0136f09 Compare August 3, 2022 01:49
This is consistent with the BaseMaterial3D filtering options.
It can be used for high-quality pixel art textures that remain sharp
when viewed at oblique angles, but prevents them from becoming grainy
thanks to mipmaps.
@Calinou Calinou force-pushed the decal-projector-add-nearest-mipmap-aniso branch from 0136f09 to d041ca6 Compare August 3, 2022 01:49
@Calinou
Copy link
Member Author

Calinou commented Aug 3, 2022

Rebased and tested again, it works as expected.

@akien-mga akien-mga merged commit b27f065 into godotengine:master Aug 3, 2022
@akien-mga
Copy link
Member

Thanks!

@Calinou Calinou deleted the decal-projector-add-nearest-mipmap-aniso branch August 3, 2022 09:47
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