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

why the spatial denoise kernal size is 5x5? #6

Closed
ZhangRuFu opened this issue Feb 9, 2022 · 2 comments
Closed

why the spatial denoise kernal size is 5x5? #6

ZhangRuFu opened this issue Feb 9, 2022 · 2 comments

Comments

@ZhangRuFu
Copy link

thank you very much for providing this implementation and improve for GTAO :)
I‘m confused about the kernal size when i read the source in XeGTAO_Denoise function
image

Is it a 3x3 spatial kernal size actually...?

@fstrugar
Copy link
Contributor

fstrugar commented Feb 9, 2022

Hi, thanks for the kind words! In one of the last changes, the kernel was reduces from 5x5 to 3x3, but is (optionally) ran twice. When applied twice it is still effectively covering a 5x5 area :)

[note: I left Intel and am no longer maintaining this repository - hopefully Intel will find a suitable replacement]

@ZhangRuFu
Copy link
Author

Oh, I get it, thank you for your reply

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants