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

support HTC #2409

Closed
sipie800 opened this issue Sep 7, 2023 · 6 comments
Closed

support HTC #2409

sipie800 opened this issue Sep 7, 2023 · 6 comments

Comments

@sipie800
Copy link

sipie800 commented Sep 7, 2023

Motivation

We've been working around HTC in mmdet. It's doing well in our project. However after expensive training effort, when we try to deploy it it took long time for us to find that mmdeploy doesn't support it. It's quite depressing.
IMO you might write supported models in a standing out position in doc. Perhaps at first of other items.

Nevertheless, we are still struggling to figure out how to support HTC in mmdeploy. The mmdeploy does implement supported models in its own version right? What is the purpose? Can we just export any torch models to onnx and trt in reguler way without mmdeploy?

Or we also may implement HTC support in mmdeploy by self. Can you provide some clue about how can we start ?

We do know that Cascade RCNN is supported. But for a hard project, it doesn't meet our need as much as HTC.

Related resources

No response

Additional context

No response

@RunningLeon
Copy link
Collaborator

hi, welcome to support HTC and PR to us. You may need to add some rewriting functions to support torch2onnx. Could refer to this doc: https://mmdeploy.readthedocs.io/en/latest/07-developer-guide/support_new_model.html

@irexyc irexyc mentioned this issue Sep 14, 2023
@irexyc
Copy link
Collaborator

irexyc commented Sep 14, 2023

Hi @sipie800 , you can test if #2438 can solve your problem.

I test it with official model and below is my convert command.

python3 tools/deploy.py configs/mmdet/instance-seg/instance-seg_onnxruntime_dynamic.py /root/workspace/mmdetection/configs/htc/htc_r50_fpn_1x_coco.py https://download.openmmlab.com/mmdetection/v2.0/htc/htc_r50_fpn_1x_coco/htc_r50_fpn_1x_coco_20200317-7332cf16.pth /root/workspace/mmdetection/demo/demo.jpg --work-dir work/htc-onnx --dump-info

python3 tools/deploy.py configs/mmdet/instance-seg/instance-seg_tensorrt_dynamic-320x320-1344x1344.py /root/workspace/mmdetection/configs/htc/htc_r50_fpn_1x_coco.py https://download.openmmlab.com/mmdetection/v2.0/htc/htc_r50_fpn_1x_coco/htc_r50_fpn_1x_coco_20200317-7332cf16.pth /root/workspace/mmdetection/demo/demo.jpg --work-dir work/htc-trt --dump-info --device cuda

@github-actions
Copy link

This issue is marked as stale because it has been marked as invalid or awaiting response for 7 days without any further response. It will be closed in 5 days if the stale label is not removed or if there is no further response.

@github-actions github-actions bot added the Stale label Sep 22, 2023
@sipie800
Copy link
Author

Thank you. It works!

@github-actions github-actions bot removed the Stale label Sep 23, 2023
@github-actions
Copy link

This issue is marked as stale because it has been marked as invalid or awaiting response for 7 days without any further response. It will be closed in 5 days if the stale label is not removed or if there is no further response.

@github-actions github-actions bot added the Stale label Sep 30, 2023
@github-actions
Copy link

github-actions bot commented Oct 5, 2023

This issue is closed because it has been stale for 5 days. Please open a new issue if you have similar issues or you have any new updates now.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants