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 support for remaining unsupported ASTC formats #2963

Merged
merged 2 commits into from
Oct 26, 2022

Conversation

RazielXYZ
Copy link
Contributor

Add support for remaining unsupported ASTC formats
Add new ASTC formats into pixelformats example and ninja scripts
Rebuild ASTC textures

Add new ASTC formats into pixelformats example and ninja scripts
Rebuild ASTC textures
@RazielXYZ RazielXYZ requested a review from bkaradzic as a code owner October 25, 2022 23:37
@bkaradzic
Copy link
Owner

Update IDL...

Bump version here:
https://github.com/bkaradzic/bgfx/blob/master/scripts/bgfx.idl#L4

Add formats here:

enum.TextureFormat { comment = "Texture formats:" }

Then run genie idl in scripts directory.

, "textures/texture_compression_astc_6x6.dds"
, "textures/texture_compression_astc_8x5.dds"
, "textures/texture_compression_astc_8x6.dds"
, "textures/texture_compression_astc_8x8.dds"
Copy link
Owner

Choose a reason for hiding this comment

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

Flip comma placement to the end of line...

@@ -193,74 +193,82 @@ typedef enum bgfx_texture_format
BGFX_TEXTURE_FORMAT_ATC, /** (17) ATC RGB 4BPP */
BGFX_TEXTURE_FORMAT_ATCE, /** (18) ATCE RGBA 8 BPP explicit alpha */
BGFX_TEXTURE_FORMAT_ATCI, /** (19) ATCI RGBA 8 BPP interpolated alpha */
BGFX_TEXTURE_FORMAT_ASTC4X4, /** (20) ASTC 4x4 8.0 BPP */
Copy link
Owner

Choose a reason for hiding this comment

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

Once you update IDL this file will be regenerated...

Move comma in pixelformats
@RazielXYZ
Copy link
Contributor Author

Didn't know about the idl, done now!

@bkaradzic bkaradzic merged commit c2bc517 into bkaradzic:master Oct 26, 2022
mipek pushed a commit to mipek/bgfx that referenced this pull request Mar 2, 2024
* Add support for remaining unsupported ASTC formats
Add new ASTC formats into pixelformats example and ninja scripts
Rebuild ASTC textures

* Update idl and generate
Move comma in pixelformats

Co-authored-by: Raziel Alphadios <raziely@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants