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

Integration with Basis Universal GPU Texture Codec #1378

Closed
filipwiech opened this issue Jul 8, 2019 · 6 comments
Closed

Integration with Basis Universal GPU Texture Codec #1378

filipwiech opened this issue Jul 8, 2019 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@filipwiech
Copy link

filipwiech commented Jul 8, 2019

Is your feature request related to a problem? Please describe.
Minimizing textures size for on-disk storage and network transport (downloading, patching, etc), while supporting each platform's preferred compressed GPU texture format for optimal rendering performance.

Describe the solution you'd like
Integration of Filament with recently open-sourced and backed by Google Basis Texture Codec.

Describe alternatives you've considered
Compressing texture files with LZ4.

OS and backend
Thanks to Basis multiplatform support it should be transparent to end-users.

@romainguy romainguy added the enhancement New feature or request label Jul 8, 2019
@romainguy
Copy link
Collaborator

We have to think about whether it makes sense to be directly in Filament. There's technically no reason why it cannot be done in the app or as a separate library (like our glTF loader for instance).

@prideout
Copy link
Contributor

prideout commented Jul 8, 2019

Our mipgen tool can be used to compress textures and it does not yet have basis support, that's something we can consider adding.

@donmccurdy
Copy link

Just FYI, this documentation may be helpful for supporting KTX and Basis Universal: https://github.com/KhronosGroup/3D-Formats-Guidelines/blob/main/KTXDeveloperGuide.md. It's a fairly new document, please let us know if you have any issues with it.

/cc @lexaknyazev (author)

@romainguy
Copy link
Collaborator

Thanks @donmccurdy!

@romainguy
Copy link
Collaborator

@prideout Is there still more work to do here?

@prideout
Copy link
Contributor

@romainguy this work is complete, this issue is basically a dup #4771, I forgot to close these issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants