Skip to content

[Fix] Fix TensorRT 7 SSD#49

Merged
RunningLeon merged 1 commit intoopen-mmlab:masterfrom
grimoire:fix-trt7-ssd
Jan 11, 2022
Merged

[Fix] Fix TensorRT 7 SSD#49
RunningLeon merged 1 commit intoopen-mmlab:masterfrom
grimoire:fix-trt7-ssd

Conversation

@grimoire
Copy link
Copy Markdown
Member

@grimoire grimoire commented Jan 6, 2022

TensorRT 7 does not support dynamic clamp, which is used in torch.nn.functional.normalize.

@grimoire grimoire added the bug Something isn't working label Jan 6, 2022
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 6, 2022

Codecov Report

Merging #49 (aad3bac) into master (b5d6c03) will decrease coverage by 0.06%.
The diff coverage is 25.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #49      +/-   ##
==========================================
- Coverage   67.67%   67.61%   -0.07%     
==========================================
  Files         172      172              
  Lines        5677     5685       +8     
  Branches      903      904       +1     
==========================================
+ Hits         3842     3844       +2     
- Misses       1554     1560       +6     
  Partials      281      281              
Flag Coverage Δ
unittests 67.61% <25.00%> (-0.07%) ⬇️

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

Impacted Files Coverage Δ
mmdeploy/codebase/mmdet/models/necks.py 41.66% <25.00%> (-33.34%) ⬇️

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 b5d6c03...aad3bac. Read the comment docs.

Copy link
Copy Markdown
Collaborator

@RunningLeon RunningLeon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@SingleZombie SingleZombie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@RunningLeon RunningLeon merged commit d1528e5 into open-mmlab:master Jan 11, 2022
@grimoire grimoire deleted the fix-trt7-ssd branch May 7, 2022 06:31
hanrui1sensetime pushed a commit to hanrui1sensetime/mmdeploy that referenced this pull request Nov 25, 2022
* fix yolov5_description.md

* add Dockerfile
ewernn pushed a commit to ewernn/mmdeploy that referenced this pull request Nov 17, 2025
* first

* fix0

* fix1

* dirty work

* wip

* add allocator

* finally done!

* lint

* fix lint

* better gather

* better onnx2ncnn

* fix expand

* [Fix] NCNN TensorSlice op bugs (open-mmlab#42)

* fix custom ops support, fix multiple mark bug, add name mapping

* check if the value_info need to be added

* remove unnecessary print

* add nms implement

* two stage split wip

* add two stage split

* add split retinanet visualize

* add two stage split (wip)

* finish two stage split

* fix lint

* move parse string to mmdeploy.utils

* add calib data generator

* create calib dataset

* finish end2end int8

* add split two stage tensorrt visualize

* fix tensorslice bugs

* fix lint

* fix clang-format

* remove comments

* int param

* fix lint

Co-authored-by: grimoire <yaoqian@sensetime.com>

* add two stage ncnn support

* remove unused ops

* git unused config

* remove no_grad, should add in refactor

* add ncnn wrapper

* fix lint

* size return tuple

* Resolve grammar error

* Fix lint

* Trim Trailing Whitespace

* fix trim

* update wrapper

* remove logs

* remove

* csrc optimize

Co-authored-by: hanrui1sensetime <83800577+hanrui1sensetime@users.noreply.github.com>
ewernn pushed a commit to ewernn/mmdeploy that referenced this pull request Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants