-
Notifications
You must be signed in to change notification settings - Fork 666
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
mlefrancois/rt_reflection #274
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome sample. Thank you very much for contributing this :)
Runs fine on my RTX 2060.
CI is currently failing due to the copyright check. Judging by the CI log you need to update the copyright to 2021 in the CMakeLists.txt
of your sample to get all CI stages working.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the high quality pull request! This looks like a great sample of high quality.
{ | ||
int objId = gl_InstanceCustomIndexEXT; | ||
|
||
int mat_idx = mat_index[objId].i[gl_PrimitiveID]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might seem like objId
here and below (materials[objId]
) should be surrounded by nonuniformEXT
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, sorry I missed this while translating.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Runs well here lgtm;
Works fine for me on an RTX 2060 on Windows 10. The readme is a great addition too 👍 |
tested and works here. NVidia AMD Linux and Windows. |
Hi folks, I'd like to get our final approval on this one so we can release it (been stuck for a few weeks). Please help review if able. Thanks! mcj |
Description
Render a simple scene showing the basics of ray tracing, including reflection and shadow rays. The sample creates some geometries and create a bottom acceleration structure for each, then make instances of those, using different materials and placing them at different locations.
Test Environment
General Checklist:
Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist: