Skip to content

Support astc/eac/etc2/pvrtc formats#47

Merged
Roughsketch merged 8 commits intoRoughsketch:masterfrom
AllenDang:master
Oct 19, 2025
Merged

Support astc/eac/etc2/pvrtc formats#47
Roughsketch merged 8 commits intoRoughsketch:masterfrom
AllenDang:master

Conversation

@AllenDang
Copy link
Contributor

No description provided.

- Add CompressionFamily enum to group related compression algorithms
  - Add compression_family(), is_block_compressed(), container_format()
methods
  - Enable querying ETC/BC/PVRTC compression across DDS/PowerVR/PKM
containers
  - Maintain full backward compatibility with existing API
@Roughsketch
Copy link
Owner

I am a bit busy this week but will try to find time to look into these formats and review when I can. Thanks for the PR!

@Roughsketch
Copy link
Owner

Just an update, I have reviewed some of the files already and will setup a fuzzer when I get a chance. However, I have become very busy with personal stuff and can't find a solid time to deep dive into all these formats. It would help if you can provide documentation on the file formats you're adding to save me some time in having to find them to verify.

Rest assured I haven't forgotten about this. I'm hoping I can chip away at it as time allows.

Copy link
Owner

Choose a reason for hiding this comment

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

This doesn't seem to be used anywhere

Copy link
Owner

@Roughsketch Roughsketch left a comment

Choose a reason for hiding this comment

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

Besides that one stray png this looks fine to me. Please remove or use if it was meant for a test and I'll get this approved.

Thanks again for the PR and sorry it took so long to review.

@Roughsketch Roughsketch self-requested a review October 19, 2025 01:51
@Roughsketch Roughsketch merged commit ce9eb0f into Roughsketch:master Oct 19, 2025
1 check passed
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.

2 participants