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 LIIF. #227

Merged
merged 8 commits into from
Apr 6, 2021
Merged

Add LIIF. #227

merged 8 commits into from
Apr 6, 2021

Conversation

Yshuo-Li
Copy link
Collaborator

Add LIIF model for single image super-resolution.
Paper: Learning Continuous Image Representation with Local Implicit Image Function
Ref repo: https://github.com/yinboc/liif.git

@codecov
Copy link

codecov bot commented Mar 17, 2021

Codecov Report

Merging #227 (45a3d54) into master (2827d93) will decrease coverage by 0.38%.
The diff coverage is 65.97%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #227      +/-   ##
==========================================
- Coverage   81.79%   81.41%   -0.39%     
==========================================
  Files         152      153       +1     
  Lines        7223     7413     +190     
  Branches     1066     1088      +22     
==========================================
+ Hits         5908     6035     +127     
- Misses       1191     1247      +56     
- Partials      124      131       +7     
Flag Coverage Δ
unittests 81.41% <65.97%> (-0.39%) ⬇️

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

Impacted Files Coverage Δ
mmedit/models/restorers/liif.py 65.07% <65.07%> (ø)
mmedit/datasets/pipelines/utils.py 94.64% <100.00%> (+5.35%) ⬆️
mmedit/models/restorers/__init__.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2827d93...45a3d54. Read the comment docs.

@Yshuo-Li Yshuo-Li requested a review from innerlee March 30, 2021 02:46
@innerlee
Copy link
Contributor

I will take a look tonight :)

@innerlee
Copy link
Contributor

There are some thing wrong with commits.

# loss
self.pixel_loss = build_loss(pixel_loss) if pixel_loss else None

self.step_counter = 0 # counting training steps
Copy link
Contributor

Choose a reason for hiding this comment

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

What if the model is resumed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This parameter (step_counter) is actually not used and has been deleted.

Returns:
dict: Returned output.
"""
# print(data_batch.keys())
Copy link
Contributor

Choose a reason for hiding this comment

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

remove debuggin codes

@innerlee innerlee merged commit 3acb28a into open-mmlab:master Apr 6, 2021
@Yshuo-Li Yshuo-Li deleted the LIIF branch April 8, 2021 08:17
Yshuo-Li added a commit to Yshuo-Li/mmediting that referenced this pull request Jul 15, 2022
* Add LIIF.

* Add LIIF.

* modify super init.

* Add licence,optimize code.

* remove debugging codes.

* Modify __init__() function.

* Modify test function of LIIF.

Co-authored-by: 李尹硕 <SENSETIME\liyinshuo@cn0014004493l.domain.sensetime.com>
Co-authored-by: liyinshuo <liyinshuo@sensetime.com>
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.

None yet

2 participants