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

[Feature] Texture compression #237

Closed
3 of 4 tasks
Tracked by #226
kanerogers opened this issue Jul 5, 2022 · 4 comments · Fixed by #249
Closed
3 of 4 tasks
Tracked by #226

[Feature] Texture compression #237

kanerogers opened this issue Jul 5, 2022 · 4 comments · Fixed by #249
Assignees

Comments

@kanerogers
Copy link
Collaborator

kanerogers commented Jul 5, 2022

Background

At present we're texture bound for most things in Hotham. Let's fix that.

TODO

  • Determine best formats
  • Implement KTX2 support for glTF files
  • Implement for test assets
  • Profile
@kanerogers kanerogers self-assigned this Jul 5, 2022
@kanerogers
Copy link
Collaborator Author

@kanerogers kanerogers changed the title Texture compression [Feature] Texture compression Jul 5, 2022
@kanerogers
Copy link
Collaborator Author

From https://developer.qualcomm.com/sites/default/files/docs/adreno-gpu/developer-guide/gpu/best_practices_texture.html

The following strategy is recommended for selecting a texture compression format:

  1. Use ASTC compression if it is available.
  2. Otherwise, use ETC2 compression if available.
  3. Otherwise, select the compression format as follows:
    a. ATC if using alpha
    b. ETC if not using alpha

Let's give that a go!

@kanerogers
Copy link
Collaborator Author

kanerogers commented Jul 6, 2022

Right, a bit of fuckery is going to be required here:

@kanerogers
Copy link
Collaborator Author

kanerogers commented Jul 11, 2022

This is starting to look really, really good!

Remaining steps:

  • Benchmark current implementation
  • Finish off texture swizzling

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 a pull request may close this issue.

1 participant