Lachei / VulkanPBRT

Vulkan physically based raytracer including denoising
MIT License
28 stars 11 forks source link

Material color properties are not properly loaded #25

Closed Lachei closed 2 years ago

Lachei commented 2 years ago

Today i discovered that material properties like a diffuse color factor is not properly parsed by the vsgXchange assimp parser. Reason is, that the get function for template objects needs the material property one wants to get the data from to be aiPTI_Buffer. However for standard colors this is aiPTI_Float. I have implemented a workaround such that the get function for a float array is used in commit 683340c5833ccec579d22790a30c3fedd0ea59a8. We should however first get #17 merged, as otherwise we would have to fix this bug twice.

Lachei commented 2 years ago

Will be fixed with #31 .