Skip to content

Conversation

@yongwww
Copy link
Member

@yongwww yongwww commented Aug 20, 2020

@masahi @alexwong @kevinthesun @zhiics please help take a look

@yongwww
Copy link
Member Author

yongwww commented Aug 20, 2020

@masahi have incorporated your comments, please take another look, thanks!

Copy link
Member

@zhiics zhiics 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
Contributor

@kevinthesun kevinthesun left a comment

Choose a reason for hiding this comment

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

LGTM

@ZihengJiang ZihengJiang added the status: need update need update based on feedbacks label Aug 21, 2020
@yongwww
Copy link
Member Author

yongwww commented Aug 24, 2020

@masahi please take another look

@masahi masahi merged commit 37cbbd7 into apache:master Aug 24, 2020
@masahi
Copy link
Member

masahi commented Aug 24, 2020

Thanks @yongwww @kevinthesun @zhiics

@masahi masahi mentioned this pull request Aug 24, 2020
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Aug 26, 2020
* [Relay] Support for PyTorch Non-Maximum Suppression

* fix comment

* add verify_model_vm
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Aug 26, 2020
* [Relay] Support for PyTorch Non-Maximum Suppression

* fix comment

* add verify_model_vm
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Aug 26, 2020
* [Relay] Support for PyTorch Non-Maximum Suppression

* fix comment

* add verify_model_vm
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Sep 2, 2020
* [Relay] Support for PyTorch Non-Maximum Suppression

* fix comment

* add verify_model_vm
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Sep 3, 2020
* [Relay] Support for PyTorch Non-Maximum Suppression

* fix comment

* add verify_model_vm
@yongwww yongwww deleted the nms_pt branch December 8, 2020 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: need update need update based on feedbacks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants