mmdetection icon indicating copy to clipboard operation
mmdetection copied to clipboard

[Feature] utilize mmcv image backend

Open fcakyon opened this issue 3 years ago • 5 comments

Currently LoadImageFromFile pipeline doesnt support loading mmcv image backends as pillow,tifffile... but mmcv supports it.

This PR adds the support of using other mmcv image backends.

fcakyon avatar Jan 23 '22 14:01 fcakyon

Codecov Report

Patch coverage is 100.00% of modified lines.

Files Changed Coverage
mmdet/datasets/pipelines/loading.py 100.00%

:loudspeaker: Thoughts on this report? Let us know!.

codecov[bot] avatar Jan 23 '22 15:01 codecov[bot]

Hi @fcakyon , Thanks for your kind PR. We plan to merge this PR this month, do you have time to continue on this?

ZwwWayne avatar Feb 10 '22 07:02 ZwwWayne

Hi @fcakyon , Thanks for your kind PR. We plan to merge this PR this month, do you have time to continue on this?

unfortunately i cannot fix the pre-commit error

fcakyon avatar Feb 10 '22 18:02 fcakyon

Maybe you should also add some code in init function: if self.image_backend is not None: mmcv.use_backend(self.image_backend) if not, you can not use turbojpeg as backend. jpeg

However, now, using turbojpeg as backend may have a potiential bug in distributing training and test as pointed in jpeg is None

JingweiZhang12 avatar Feb 11 '22 11:02 JingweiZhang12

@fcakyon Do you have any progress?

hhaAndroid avatar Mar 09 '22 11:03 hhaAndroid