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

Add nl means denoising open cv in image processing node #1719

Merged
merged 3 commits into from
Dec 13, 2022

Conversation

tanstella
Copy link

@tanstella tanstella commented Jun 29, 2022

Description

Added parameters in ImageProcessing node for denoising by OpenCV (NLMeans denoising)

Based on alicevision/AliceVision#1181

Copy link
Member

@fabiencastan fabiencastan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to delete stale files from git.

@tanstella tanstella force-pushed the AddNLMeansDenoisingOpenCVInImageProcessingNode branch from 2e87824 to aaa8504 Compare August 10, 2022 09:53
@tanstella
Copy link
Author

Done

@cbentejac cbentejac force-pushed the AddNLMeansDenoisingOpenCVInImageProcessingNode branch from aaa8504 to 85a3f2a Compare November 30, 2022 16:42
…default value

Change HColor default value from 5 to 10 as advised in the documentation
of OpenCV's non-local means denoising. Also update the description of the
H and HColor parameters so that they are easier to understand. The name
of the parameters is also updated to match with parameters from other
groups.
@cbentejac cbentejac force-pushed the AddNLMeansDenoisingOpenCVInImageProcessingNode branch from 098f4dc to 826b57e Compare December 5, 2022 14:58
@fabiencastan fabiencastan added this to the Meshroom 2022.1.0 milestone Dec 12, 2022
@fabiencastan fabiencastan merged commit 82ae95e into develop Dec 13, 2022
@fabiencastan fabiencastan deleted the AddNLMeansDenoisingOpenCVInImageProcessingNode branch December 13, 2022 21:48
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

Successfully merging this pull request may close these issues.

3 participants