RenderKit / oidn

Intel® Open Image Denoise library
https://www.openimagedenoise.org/
Apache License 2.0
1.78k stars 164 forks source link

Flexible build with filter selection #86

Closed WilliamTambellini closed 3 years ago

WilliamTambellini commented 4 years ago

Hi Some softwares are not always interested by all OIDN filters. In such case, disabling some filters allows to significantly reduce the size of the library and some memory considering that the models are not loaded from disk at runtime but embedded in the lib (release/libOpenImageDenoise.so.1.2.2) :

Would you mind if I prepare a PR to add new cmake option(s) on order to (either):

Ref: https://github.com/godotengine/godot/blob/master/thirdparty/oidn/patches/godot-changes-c58c5216.patch

Kind W.

atafra commented 4 years ago

Hi,

Thanks for the feedback! We'll implement this feature in the next major version (1.3.0).

Regards, Attila

WilliamTambellini commented 4 years ago

Cool, tks.

WilliamTambellini commented 3 years ago

@atafra Hi, has that one been included in 1.3.0: https://github.com/OpenImageDenoise/oidn/blob/master/CHANGELOG.md ?

atafra commented 3 years ago

No, sorry, this feature has been postponed for the next release (which should be available sometime in March). Since v1.3.0 there are multiple sets of weights for RTLightmap as well. Is it still sufficient for you to just disable the RT filter and keep both blobs for RTLightmap? Or do you need a more fine-grained control over each weight blob?

atafra commented 3 years ago

This feature is now implemented in v1.4.0.