fix compression after resizing in remote inference

[jianwensong]

While testing remote inference with compress_after_resizing enabled, I found that the current implementation ignores the output of _org_mapper_func.

The resized images should all be in RGB format to ensure correct YUV conversion. However, in both YOLOXCustomMapper, Detectron2’s DatasetMapper, and MMPOSECustomMapper, the output of _org_mapper_func is in BGR format. Therefore, using these outputs directly leads to incorrect colors when compress_after_resizing is enabled. This PR provides a fix to this issue.

In addition, a minor issue related to image-based remote inference when enabling compress_after_resizing has been fixed. (f6d23ae has fixed this, the conflict has been solved) Could you please review the fix? Thanks.

[fracape]

Hi Jianwen,
Apologies, I forgot to push Hyomin's PR on this to the public github. I see that you are rebasing. We'll check tomorrow (our time)

[jianwensong]

Hi Jianwen, Apologies, I forgot to push Hyomin's PR on this to the public github. I see that you are rebasing. We'll check tomorrow (our time)

No problem. I have rebased the code and solved the conflict. Thanks.

[fracape]

Did you close it? I believe you address a bit more than what was done.

[jianwensong]

Sorry. I intended to delete one comment and wrongly selected close the PR.

[chyomin06]

@jianwensong , Thanks for the PR.
I reviewed the changes, it is agreed to have a proper conversion to RGB format to save an image.
However, it is preferable to minize changes since it may affect the original path.
Please review the latest commit with hash 8305e3e , and let us know if the updates work for you.

[jianwensong]

@chyomin06 Thanks for the updates. That looks really good! The only issue is that this commit seems not work with Detectron2’s DatasetMapper (the one imported directly from detectron2, not the customized version in utils), which I believe also outputs images in BGR format.

[chyomin06]

@jianwensong , Thanks for the PR. I reviewed the changes, it is agreed to have a proper conversion to RGB format to save an image. However, it is preferable to minize changes since it may affect the original path. Please review the latest commit with hash 8305e3e , and let us know if the updates work for you.

I think it should work since Detectron2 dataset mapper has the same property attribute, image_format.

[jianwensong]

@chyomin06 I see. Thank you for the explanation. Yes, you are right. I will close this PR.