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 DnCNN with noise level input #349

Merged
merged 32 commits into from
Dec 12, 2022
Merged

Conversation

wjgancn
Copy link
Contributor

@wjgancn wjgancn commented Oct 3, 2022

Add DnCNN with noise level input.

@codecov
Copy link

codecov bot commented Oct 3, 2022

Codecov Report

Merging #349 (2cb70af) into main (fd26f49) will decrease coverage by 0.05%.
The diff coverage is 95.00%.

@@            Coverage Diff             @@
##             main     #349      +/-   ##
==========================================
- Coverage   94.15%   94.10%   -0.06%     
==========================================
  Files          57       59       +2     
  Lines        3406     3578     +172     
==========================================
+ Hits         3207     3367     +160     
- Misses        199      211      +12     
Flag Coverage Δ
unittests 94.10% <95.00%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
scico/denoiser.py 88.57% <95.00%> (+0.79%) ⬆️
scico/__init__.py 100.00% <0.00%> (ø)
scico/util.py 92.46% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@crstngc
Copy link
Contributor

crstngc commented Oct 4, 2022

I would suggest to use N as the suffix to denote the noise-conditional models instead of V. Let me ask for other suggestions before proceeding with any change.

@crstngc crstngc requested a review from bwohlberg October 4, 2022 16:57
examples/scripts/denoise_dncnn_with_noise_level_input.py Outdated Show resolved Hide resolved
examples/scripts/denoise_dncnn_with_noise_level_input.py Outdated Show resolved Hide resolved
examples/scripts/denoise_dncnn_with_noise_level_input.py Outdated Show resolved Hide resolved
examples/scripts/README.rst Outdated Show resolved Hide resolved
@bwohlberg bwohlberg added the enhancement New feature or request label Oct 20, 2022
@wjgancn
Copy link
Contributor Author

wjgancn commented Oct 29, 2022

I have revised my code based on your suggestions.

Especially, I rewrote the DnCNN denoiser class, which is a bit large change from my viewpoint. Maybe having a new class of denoiser is a better idea, but it also comes with code redundancy.

@bwohlberg bwohlberg merged commit 2e64c7d into lanl:main Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants