KTX-Software icon indicating copy to clipboard operation
KTX-Software copied to clipboard

Published 20 hours ago • verified publisher icon KhronosGroup

Flesh out support for VK_FORMAT_R16G16_S10_5_NV.

Open MarkCallow opened this issue 11 months ago • 1 comments

Code is believed complete. Tests have yet to be added. Hence draft status.

This includes a new round-trip test of vk2dfd and dfd2vk for all Vulkan formats except multi-plane and modifications to both functions, or rather their generators, to fix support for some formats, including VK_FORMAT_R16G16_S10_5_NV and the YUV 422 single plane formats. Those listed required changes tointerpretDFD.

Fixes #859.

MarkCallow avatar Mar 01 '24 03:03 MarkCallow

@fluppeteer, please review. I was unable to add your GitHub id to the reviewers list.

MarkCallow avatar Mar 01 '24 03:03 MarkCallow

I think this mostly looks good, except that the multi-channel packing convention wires got crossed. I've added notes where I think the numbers should be different. Please poke me if you change them and want a re-review, or want to push back; I'll try to respond promptly this time.

fluppeteer avatar Mar 16 '24 19:03 fluppeteer

The #define approach looks much easier to read (not a phrase I'll often say) - thanks for clarifying, and sorry again to have caused the churn!

Assuming all the test are happy, I think this is good to go, and we can talk about coordinate offsets separately.

fluppeteer avatar Mar 19 '24 21:03 fluppeteer