Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@DGriffin91
Copy link
Contributor

Added support for block formats in format_texel_block_size(), copy_buffer_to_image_regions(), copy_image_to_buffer_regions(). Added format_texel_block_dimensions().

Added remaining formats in https://registry.khronos.org/vulkan/specs/latest/html/vkspec.html#texel-block-size

Have only tested with BCn textures.

@DGriffin91
Copy link
Contributor Author

Looks like this test driver::image::tests::image_info_cube_builder also fails on master.

@attackgoat
Copy link
Owner

I broke ImageInfo tests while trying to fix an issue I ran into doing gpu-culled depth pyramid cubes for point shadow rendering. The root issue was that Screen-13 has invented a new way of describing cube images that differs from the spec, and while that might make it easier to understand these images without prior knowledge, it greatly complicates any understanding that you bring to Screen-13 or while adapting other examples.

The fix, although it will take a new version, is to finally remove screen_13::driver::image::ImageType and let users rely on vk::ImageType. I'll push this branch soon.

I only mention that because I would like to make this crate easier to use and I'm finding it's the little things that impede this goal. To that end, could we rename format_texel_block_dimensions -> format_texel_block_extent?

This change LGTM.

@attackgoat attackgoat merged commit fcd45f8 into attackgoat:master May 12, 2025
2 of 3 checks 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