[Enhance] Support non-zero frame index for SRFolderVideoDataset & Fix Typos

[ryanxingql]

Polish

1. Correct "annoations" to "annotations".
2. Correct "propgation" to "propagation".
3. Remove undefined functions in mmedit/datasets/pipelines/__init__.py.

Improvement

Some sequences may start from a frame index other than 0.
If without annotation files, SRFolderVideoDataset cannot process these sequences correctly; sequences must start from 0.
But in many cases, we use our own data-sets to train video SR methods such as EDVR, and the sequences in these data-sets may start from a frame index other than 0.

[codecov]

Codecov Report

Merging #853 (9e8fb9a) into master (a023cdf) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #853      +/-   ##
==========================================
+ Coverage   83.10%   83.12%   +0.01%     
==========================================
  Files         220      220              
  Lines       12452    12453       +1     
  Branches     2017     2017              
==========================================
+ Hits        10348    10351       +3     
+ Misses       1786     1785       -1     
+ Partials      318      317       -1     

Flag	Coverage Δ
unittests	83.08% <100.00%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
mmedit/datasets/comp1k_dataset.py	100.00% <ø> (ø)
mmedit/datasets/pipelines/__init__.py	100.00% <ø> (ø)
mmedit/datasets/sr_annotation_dataset.py	100.00% <ø> (ø)
mmedit/datasets/sr_facial_landmark_dataset.py	100.00% <ø> (ø)
mmedit/datasets/sr_folder_dataset.py	100.00% <ø> (ø)
mmedit/datasets/sr_folder_gt_dataset.py	100.00% <ø> (ø)
mmedit/datasets/sr_folder_multiple_gt_dataset.py	100.00% <ø> (ø)
mmedit/datasets/sr_folder_ref_dataset.py	89.18% <ø> (ø)
mmedit/datasets/sr_lmdb_dataset.py	100.00% <ø> (ø)
mmedit/datasets/sr_reds_dataset.py	100.00% <ø> (ø)
... and 10 more

---

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 a023cdf...9e8fb9a. Read the comment docs.

[wangruohui]

Hi,

It seems the linter fails. Would you please format the file you modified using yapf?
You can use that in vscode or pycharm.

Another option is to configure pre-commit hook following this

[ryanxingql]

Hi @wangruohui , thanks for your response!

I have tried to polish my file through yapf, but nothing changed. Can you give me some suggestions?

Thank you!

PS the problematic commit id is: 88ea442
I have tried to polish the modified file mmedit/datasets/sr_folder_video_dataset.py.

[ryanxingql]

@wangruohui Thank you so much!

[wangruohui]

Hello @ryanxingql ,

Still one more thing to de before merging. As you have modified the argument list, you need to add a docstring to indicate its usage for future users. Just add a description around line 54-55.

Thank you!

[ryanxingql]

Hello @ryanxingql ,

Still one more thing to de before merging. As you have modified the argument list, you need to add a docstring to indicate its usage for future users. Just add a description around line 54-55.

Thank you!

No problem! I have added the docstring. Please check.