OpenImageDenoise

Discussion related to the LuxCore functionality, implementations and API.
CodeHD
Donor
Donor
Posts: 437
Joined: Tue Dec 11, 2018 12:38 pm
Location: Germany

Re: OpenImageDenoise

Post by CodeHD »

How about calling it "Mix (with) Undenoised" in Blender? I would think if a user selects a denoiser, he expects a denoised image by default. Hence name it something that explicitly states how it deviates from pure denoised image.

In LuxCore itself (the parameter Dade mentions) could just be "mix" for simplicity?
User avatar
B.Y.O.B.
Developer
Developer
Posts: 4146
Joined: Mon Dec 04, 2017 10:08 pm
Location: Germany
Contact:

Re: OpenImageDenoise

Post by B.Y.O.B. »

For consistency's sake, I would call the property "amount".
This is how a simple lerp is called in the mix texture, mix material etc.
In Blender, lerps are called "fac" everywhere, which is probably short for "factor".

However we call it, I think (almost) every imagepipeline plugin should have this parameter, to make it easier to control the strength of it.
juangea
Donor
Donor
Posts: 332
Joined: Thu Jan 02, 2020 6:23 pm

Re: OpenImageDenoise

Post by juangea »

Yep, in Blender Fac. is Factor.

And "amount" could be fine if it has consistency with the rest of the UI, as long as the effect of this parameter is properly explained in the documentation there shouldn`t be any problem I think :)
zuljin3d
Posts: 76
Joined: Sun Apr 08, 2018 12:13 pm
Location: Moscow

Re: OpenImageDenoise

Post by zuljin3d »

Open Image Denoise v1.2.0

-Added neural network training code
-Added support for specifying user-trained models at runtime
-Slightly improved denoising quality (e.g. less ringing artifacts, less
blurriness in some cases)
-Improved denoising speed by about 7-38% (mostly depending on the compiler)
-Added OIDN_STATIC_RUNTIME CMake option (for Windows only)
-Added support for OpenImageIO to the example apps (disabled by default)
-Added check for minimum supported TBB version
-Find debug versions of TBB
-Added testing
https://github.com/OpenImageDenoise/oidn/releases
Actualy sorry for my google translate english :)
Post Reply