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 sr_folder_ref_dataset #292

Merged
merged 6 commits into from
May 21, 2021
Merged

add sr_folder_ref_dataset #292

merged 6 commits into from
May 21, 2021

Conversation

Yshuo-Li
Copy link
Collaborator

@Yshuo-Li Yshuo-Li commented May 7, 2021

For TTSR

@codecov
Copy link

codecov bot commented May 7, 2021

Codecov Report

Merging #292 (31beb6c) into master (ee2d010) will increase coverage by 0.03%.
The diff coverage is 89.47%.

❗ Current head 31beb6c differs from pull request most recent head 1ec92d6. Consider uploading reports for the commit 1ec92d6 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
+ Coverage   80.93%   80.97%   +0.03%     
==========================================
  Files         164      165       +1     
  Lines        8228     8266      +38     
  Branches     1194     1200       +6     
==========================================
+ Hits         6659     6693      +34     
  Misses       1425     1425              
- Partials      144      148       +4     
Flag Coverage Δ
unittests 80.97% <89.47%> (+0.03%) ⬆️

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

Impacted Files Coverage Δ
mmedit/datasets/sr_folder_ref_dataset.py 89.18% <89.18%> (ø)
mmedit/datasets/__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 ee2d010...1ec92d6. Read the comment docs.

mmedit/datasets/sr_folder_ref_dataset.py Outdated Show resolved Hide resolved
mmedit/datasets/sr_folder_ref_dataset.py Outdated Show resolved Hide resolved
mmedit/datasets/sr_folder_ref_dataset.py Outdated Show resolved Hide resolved
mmedit/datasets/sr_folder_ref_dataset.py Show resolved Hide resolved
tests/test_datasets.py Show resolved Hide resolved
@@ -72,6 +73,7 @@ def __init__(self,
filename_tmp_gt='{}',
filename_tmp_lq='{}'):
super().__init__(pipeline, scale, test_mode)
assert gt_folder or lq_folder
Copy link
Member

Choose a reason for hiding this comment

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

Better add a message here (e.g. At least one of 'gt_folder and 'lq_folder' cannot be None.). Looks good to me other than this.

@Yshuo-Li Yshuo-Li requested a review from innerlee May 18, 2021 01:53
@innerlee innerlee merged commit b6b6838 into open-mmlab:master May 21, 2021
@Yshuo-Li Yshuo-Li deleted the ref branch May 21, 2021 23:17
Yshuo-Li added a commit to Yshuo-Li/mmediting that referenced this pull request Jul 15, 2022
* add sr_folder_ref_dataset

* add test

* Fix

* Fix

* Fix

* Rename

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

3 participants