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

Add TensorRT Support and YOLOv5 #100

Merged
merged 13 commits into from
Dec 29, 2022
Merged

Add TensorRT Support and YOLOv5 #100

merged 13 commits into from
Dec 29, 2022

Conversation

stefanopini
Copy link
Owner

@stefanopini stefanopini commented Nov 27, 2022

Add TensorRT Support from #99 (in branch tensorrt) to master branch.

  • Fix folder structure
  • Clean up code
  • Update docstrings
  • Add missing params
  • Test on Colab
  • Add documentation
  • Add colab link

gpastal24 and others added 5 commits November 27, 2022 17:45
* [Add] Yolov5 trt HRNet inference on video

* Fixed a bug with int overflow on tracking

* Revisions before review

Co-authored-by: Giannis Pastaltzidis <[email protected]>
@stefanopini stefanopini changed the title Add TensorRT Support Add TensorRT Support and YOLOv5 Dec 1, 2022
@gpastal24
Copy link
Contributor

Hi @stefanopini , just checking in. It has come to my attention that there is a new yolov5 release(version 7). When I tried to convert it to trt with the latest version I think there was an error. version v6.2 worked fine if i remember correctly and it can be loaded by adding :v6.2 in the torch hub str.

Anyways, I will have free time and I can handle some tasks before the merge, next week if you would like to.

@stefanopini
Copy link
Owner Author

Hi @gpastal24 , thanks for the update! I've been trying the code on a colab notebook and I should be able to finish working on it soon. I'll try the yolov5 release 7 and switch to the 6.2 if not working.

If you have some time this week, I'll ask you to take a look at the code and test the notebook as soon as they're ready, thank you!

@stefanopini
Copy link
Owner Author

Hi @gpastal24 , I managed to test the new code and to make the conversion to tensorrt work with the latest yolov5 too! Could you give it a try too?
You can open the notebook in google colab directly using this link.

@gpastal24
Copy link
Contributor

Hello @stefanopini , I tried the notebook . It worked great

@stefanopini
Copy link
Owner Author

Amazing, thanks for checking! Going to merge soon

@stefanopini stefanopini merged commit 6bfcdaf into master Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants