X Tutup
Skip to content

Image: Support generating mipmaps for all uncompressed formats#110257

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
BlueCube3310:mip-gen-all-formats
Oct 1, 2025
Merged

Image: Support generating mipmaps for all uncompressed formats#110257
Repiteo merged 1 commit intogodotengine:masterfrom
BlueCube3310:mip-gen-all-formats

Conversation

@BlueCube3310
Copy link
Contributor

@BlueCube3310 BlueCube3310 commented Sep 4, 2025

Supports generating mipmaps for RGB565 and RGBA4444.

TODO:

  • 4 new inline functions were required to convert between Color and the respective format. Should they be moved to the Color class instead?

@BlueCube3310 BlueCube3310 requested a review from a team as a code owner September 4, 2025 16:23
@BlueCube3310 BlueCube3310 added this to the 4.x milestone Sep 4, 2025
@BlueCube3310 BlueCube3310 force-pushed the mip-gen-all-formats branch 2 times, most recently from 7c840fd to d0ae84d Compare September 9, 2025 19:43
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

Tested locally, it works as expected. Code looks good to me.

Testing project: test_image_mipmaps.zip

Note that on RGB565, mipmaps will sometimes bias towards green or purple for similar reasons as described in this video. There's not much we can do about it. In the this example, the pixels should average to a neutral gray but it looks a bit green instead:

image

The same texture in RGB8 for comparison:

image

@Repiteo Repiteo modified the milestones: 4.x, 4.6 Sep 10, 2025
@Repiteo
Copy link
Contributor

Repiteo commented Sep 24, 2025

Needs rebase

@Repiteo Repiteo merged commit f9be832 into godotengine:master Oct 1, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Oct 1, 2025

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

X Tutup