Working inside linux #12
Conversation
|
|
|
well as i got through gdb that this project is using nativefiledialog, which currently doesnt support wayland so people using wayland might have problem here |
README.md
Outdated
| @@ -207,7 +207,7 @@ Add the following CMake user preset file in your project directory. I'll assume | |||
| "inherits": "default", | |||
| "cacheVariables": { | |||
| "CMAKE_C_COMPILER": "/usr/bin/clang-18", | |||
There was a problem hiding this comment.
Could you also change clang-18 to clang?
| [^3]: On Apple GPU platform prior to the MoltenVK 1.2.11 (which enables the Metal Argument Buffer by default), [`maxPerStageDescriptorUpdateAfterBindStorageImages` is 8](https://vulkan.gpuinfo.org/displaycoreproperty.php?platform=macos&name=maxPerStageDescriptorUpdateAfterBindStorageImages&core=1.2). It limited the cubemap resoluton and prefilteredmap roughnesslevels. Instead, it can use `VK_AMD_shader_image_load_store_lod` extension to replace the descriptor indexing based cubemap mipmapping and prefilteredmap generation. |
There was a problem hiding this comment.
Why this line has been changed?
| brdfmapComputer.descriptorSetLayout.getPoolSize().getDescriptorPoolCreateInfo(), | ||
| }; | ||
|
|
||
| vk::DescriptorPoolCreateInfo descriptorPoolCreateInfo = brdfmapComputer.descriptorSetLayout.getPoolSize().getDescriptorPoolCreateInfo(); |
There was a problem hiding this comment.
This would not be work. vku::PoolSizes::getDescriptorPoolCreateInfo returns both vk::DescriptorPoolCreateInfo and std::vector<vk::DescriptorPoolSize> via vku::RefHolder (former references the latter), but store it into vk::DescriptorPoolCreateInfo will destroy the pool size vector.
You should pass vk::DescriptorPoolCreateFlagBits::eFreeDescriptorSet to the method by parameter and use the struct directly.
impl/vulkan/Frame.cpp
Outdated
| .getDescriptorPoolCreateInfo(), | ||
| }; | ||
|
|
||
| auto poolCreateInfo = |
| @@ -7,6 +7,7 @@ import :vulkan.pipeline.BrdfmapComputer; | |||
|
|
|||
| import std; | |||
|
|
|||
| #define COMPILED_SHADER_DIR "shader" | |||
There was a problem hiding this comment.
Isn't it already defined by CMakeLists.txt?
Line 194 in 713156e
There was a problem hiding this comment.
Isn't it already defined by
CMakeLists.txt?Line 194 in 713156e
yeah sorry for not being careful temporarily put it there because i was getting error there while calling binary as ./build/vk-gltf-viewer
|
I'll work for NFD in Wayland support. |
|
Fix for NFD in Wayland merged in master branch. |
PR to get it properly working on linux