'mmrotate\apis\inference.py'

[sltlls]

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.

Motivation

When using demo/huge_image_demo.py to inference large image with large gap set, it will meet the problem of memory shortage.

Modification

Change the patch_datas variable from saving all data of splited images to only saving data of splited images in one batch.

Checklist

1. Pre-commit or other linting tools are used to fix the potential lint issues.
2. The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
3. The documentation has been modified accordingly, like docstring or example tutorials.

[zytx121]

@jbwang1997 Please have a look.

[codecov]

Codecov Report

Merging #368 (71969de) into main (e22c8df) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #368      +/-   ##
==========================================
- Coverage   30.86%   30.85%   -0.01%     
==========================================
  Files         119      119              
  Lines        7934     7936       +2     
  Branches     1220     1221       +1     
==========================================
  Hits         2449     2449              
- Misses       5384     5386       +2     
  Partials      101      101              

Flag	Coverage Δ
unittests	30.83% <0.00%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
mmrotate/apis/inference.py	0.00% <0.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 e22c8df...71969de. Read the comment docs.

[jbwang1997]

LGTM

[sltlls]

What does the sentence “Added line #L56 was not covered by tests” mean?

[ZwwWayne]

What does the sentence “Added line #L56 was not covered by tests” mean?

It means the unit tests do not fully test that line of code.