-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Initial VTF support #6487
base: main
Are you sure you want to change the base?
Initial VTF support #6487
Conversation
Can I provide arguments to save the function? Like compression mode, flags and etc? |
Like |
Yes, that's just what I needed, thanks! |
@radarhere Do I need to encode the image row-by-row or i can go like 4 rows at once because Bcn works in blocks of 4 by 4 pixels? |
No, you don't have to. A while ago, I attempted to implement DXT1 encoding. You might find that helpful - radarhere@363fab4 |
I see, your attempt is quite close to what I wrote, but I used a different way to get 2 main colors for the block. Can I reuse some of your code from 363fab4? |
Feel free |
@radarhere Can you please review my bcn encoder? |
Slightly confused - this PR is still a draft, but you'd like a review? Also, your lint problems could be resolved by just using the black formatting - REDxEYE#2. Is there a particular part of the styling that you dislike? |
I'm asking for a encoder code review, pr itself not even close to be ready :) |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
…use union rgb565 structs
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
This pull request adds initial support for VTF image format
Changes proposed in this pull request: