X Tutup
Skip to content

Compatibility: Improve ASTC extension detecting#109778

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
BlueCube3310:gles3-astc-detect
Sep 19, 2025
Merged

Compatibility: Improve ASTC extension detecting#109778
Repiteo merged 1 commit intogodotengine:masterfrom
BlueCube3310:gles3-astc-detect

Conversation

@BlueCube3310
Copy link
Contributor

@BlueCube3310 BlueCube3310 commented Aug 19, 2025

According to https://developer.arm.com/documentation/102162/0430/Using-ASTC-with-graphics-APIs and https://registry.khronos.org/OpenGL/extensions/OES/OES_texture_compression_astc.txt ASTC extensions are packaged in the following 'tiers':

GL_KHR_texture_compression_astc_ldr - 2d, LDR only.
GL_KHR_texture_compression_astc_sliced_3d - 2d and sliced 3d, LDR only,
GL_KHR_texture_compression_astc_hdr - 2d and sliced 3d, LDR and HDR,
GL_OES_texture_compression_astc - 2d, sliced 3d, volumetric 3d, LDR and HDR ('full'),

TODO:

  • I was unable to find any information regarding GL_KHR_texture_compression_astc, so I've decided to remove it. It has been present in the codebase since ASTC was first implemented in the engine, so it needs to be confirmed whether devices actually report it.
  • According to https://registry.khronos.org/webgl/extensions/WEBGL_compressed_texture_astc/ WEBGL_compressed_texture_astc can support both LDR and HDR profiles, and the function getSupportedProfiles may be used to query them.

@BlueCube3310 BlueCube3310 requested a review from a team as a code owner August 19, 2025 15:27
@BlueCube3310 BlueCube3310 added this to the 4.x milestone Aug 19, 2025
@fire
Copy link
Member

fire commented Aug 20, 2025

Is GL_KHR_texture_compression_astc related WEBGL with ASTC, like we have webgl with BPTC and ETC2?

@BlueCube3310
Copy link
Contributor Author

Is GL_KHR_texture_compression_astc related WEBGL with ASTC, like we have webgl with BPTC and ETC2?

WEBGL_compressed_texture_astc is the one used for WEBGL.
According to https://opengles.gpuinfo.org/listreports.php?extension=GL_KHR_texture_compression_astc GL_KHR_texture_compression_astc is only supported by 2 reported devices, so it's probably the vendors misunderstanding the extension's purpose.

@clayjohn clayjohn modified the milestones: 4.x, 4.6 Sep 2, 2025
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.

Code looks good to me.

@Repiteo Repiteo merged commit 191efe2 into godotengine:master Sep 19, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Sep 19, 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.

5 participants

X Tutup