Skip to content
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

Update headers to 1.1.91 for supporting final NV raytracing ext.. #14

Closed
oscarbg opened this issue Nov 8, 2018 · 8 comments

Comments

@oscarbg
Copy link

commented Nov 8, 2018

Hi,
new Windows 416.81 and Linux 415.13 drivers only support final NV_ray_tracing extension..
seems to be present on >=1.1.91 headers..

@zeux

This comment has been minimized.

Copy link
Owner

commented Nov 8, 2018

I am waiting for official release of Vulkan SDK to do that

@oscarbg

This comment has been minimized.

Copy link
Author

commented Nov 8, 2018

thanks!

@iOrange

This comment has been minimized.

Copy link

commented Nov 16, 2018

Had to manually fix that in volk as I'm using it to load Vulkan functions

@zeux

This comment has been minimized.

Copy link
Owner

commented Nov 16, 2018

I was hoping that the new Vulkan SDK will be released soon after the new drivers but it doesn't look like that. @iOrange I'm assuming you're using Vulkan-Headers instead of the official Vulkan SDK to get the new structures?

@iOrange

This comment has been minimized.

Copy link

commented Nov 16, 2018

Yes. It's a weird situation, where Nvidia deprecated and dropped the support for NVX, and we don't have an SDK with the new stuff. So I have to use up-to-date Vulkan Headers from github + manually built tools and compilers.

BTW - thanks for the Volk library! It's awesome!

@zeux

This comment has been minimized.

Copy link
Owner

commented Nov 16, 2018

Yeah... Basically I wanted to delay the update because I know some people used volk with NVX extensions for RTX, and updating volk would break that, but of course now with updated drivers you need new extensions.

What I'm going to do is that I'm going to update to new version of the spec, but I will leave support for NVX for now (by manually restoring that). When new SDK goes live I'll be able to just regenerate the bindings and NVX stuff will disappear.

@iOrange

This comment has been minimized.

Copy link

commented Nov 16, 2018

Thanks a lot! That'll help us. 🙇‍♂

@zeux zeux closed this in cc6ac2e Nov 16, 2018
@zeux

This comment has been minimized.

Copy link
Owner

commented Nov 16, 2018

Just submitted an update; I've checked that it seems to work with Vulkan SDK 1.1.85, and compiles with new Vulkan-Headers so in theory it should be good. It exposes NVX or NV extension based on whether the Vulkan headers in include path expose one or the other so should "just work"... hopefully :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.