DeclanRussell / NvidiaAIDenoiser

A simple implementation of Nvidia's AI denoiser
MIT License
398 stars 52 forks source link

Update OptiX to 7.6 #27

Closed Vadim-Karpenko closed 1 year ago

Vadim-Karpenko commented 1 year ago

I've used OptixDenoiserAlphaMode instead of plain int value because otherwise, it throws an error during the compilation. Not sure if this is the right thing to do, but everything works fine for me after I made this change, so I decided to create a pull request so you can check this out and decide if this is worth merging.

Vadim-Karpenko commented 1 year ago

Great idea! I completely forgot about the backward compatibility of older SDKs

DeclanRussell commented 1 year ago

Looks great, thanks a bunch! 😄

Vadim-Karpenko commented 1 year ago

Here's a new version I've compiled for myself on Windows 10, x64. I want to share it here as well so you can use it for creating a new release if you don't have a dev environment on your hands right now. Denoiser_v.2.5_unofficial.zip

mojo-jojo-7 commented 1 year ago

@Vadim-Karpenko thank you for this! Wanted to try temporal denoising but did not have the technical know-how to build from source myself.

It would be cool if a (much awaited) release is pushed with the new binaries, updated readme and everything.

ChristopheMoreau commented 1 year ago

@Vadim-Karpenko thank you for this! Wanted to try temporal denoising but did not have the technical know-how to build from source myself.

It would be cool if a (much awaited) release is pushed with the new binaries, updated readme and everything.

+1 🙏