Skip to content

Integrate qvkrt raytracing to fulldome raytracing addon #1

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

YunmaoLeo
Copy link
Owner

@YunmaoLeo YunmaoLeo commented Jul 13, 2025

This PR roughly integrate qvkrt raytracing to vfx-template.

To enable raytracing feature for vulkan, this ossia/score#1827 should be applied to ossia.

@YunmaoLeo YunmaoLeo closed this Jul 13, 2025
@YunmaoLeo YunmaoLeo reopened this Jul 13, 2025
@YunmaoLeo YunmaoLeo changed the title Integrate qvkrt raytracing to vfx-template Integrate qvkrt raytracing to fulldome raytracing addon Jul 13, 2025
@jcelerier

This comment was marked as resolved.

@YunmaoLeo
Copy link
Owner Author

YunmaoLeo commented Jul 13, 2025

@jcelerier Hi, did you patch the Qt source code and use it to build the score? I think I've met this problem when I have not correctly patch the Qt source code.

maybe try using the patch in this commit for rhi/qrhivulkan.cpp YunmaoLeo/qtbase@932dab2

@jcelerier
Copy link
Contributor

patching Qt won't do it :( we have to work with upstream Qt.

I think what we have to do is create the VkDevice ourselves with raytracing enabled and pass it to QRhi::create:

https://doc.qt.io/qt-6/qrhivulkaninitparams.html#working-with-existing-vulkan-devices

@YunmaoLeo
Copy link
Owner Author

patching Qt won't do it :( we have to work with upstream Qt.

I think what we have to do is create the VkDevice ourselves with raytracing enabled and pass it to QRhi::create:

https://doc.qt.io/qt-6/qrhivulkaninitparams.html#working-with-existing-vulkan-devices

@jcelerier Make sense! I may have been thinking this too narrowly. I will start investigate that and solve the issue in a suggested approach.

@jcelerier
Copy link
Contributor

got it to work!

ddp-2025-07-13_17.05.37.mp4

@jcelerier

This comment was marked as resolved.

@YunmaoLeo
Copy link
Owner Author

@jcelerier that's nice! will review that code.

@YunmaoLeo YunmaoLeo force-pushed the feat/integrate_qvkrt_raytracing branch from 05c2641 to 1b2e011 Compare July 15, 2025 07:51
@YunmaoLeo YunmaoLeo force-pushed the feat/integrate_qvkrt_raytracing branch from 1b2e011 to 76f83fc Compare July 15, 2025 07:55
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