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

TensorRT 8 #148

Open
auto-amc opened this issue Oct 7, 2021 · 6 comments
Open

TensorRT 8 #148

auto-amc opened this issue Oct 7, 2021 · 6 comments

Comments

@auto-amc
Copy link

auto-amc commented Oct 7, 2021

Hi,
Earlier, I tried to start my network on version TensorRT 7 and everything worked great when switching to the 8 version branch, the conversion to goes well, but the detection does not work correctly and the next error occurs the following error-
"Assertion `get3DTensorVolume(m_Engine->getBindingDimensions(tensor.bindingIndex)) == tensor.volume && "Tensor volumes dont match between cfg and engine file \n"' failed"
my config: Ubuntu 20.04 , Driver Version: 470.63.01,CUDA Version:release 11.3, V11.3.58 ,yolov4-tiny-3l.cfg from https://github.com/AlexeyAB/darknet

Thank you in advance for your reply.

@auto-amc
Copy link
Author

auto-amc commented Oct 7, 2021

ylov4 work fine,problem with yolov4-tiny

@Nuzhny007
Copy link

Tiny-yolo for TensorRT 8 (TRT8 branch) now do not works. The reason of your error is here: https://github.com/enazoe/yolo-tensorrt/blob/TRT8/modules/yolo.cpp#L1163

			if (m_NetworkType == "yolov4")//pan
			{
				outputTensor.gridSize = (m_InputH / 32) * pow(2, 2-_n_yolo_ind);
				outputTensor.grid_h = (m_InputH / 32) * pow(2, 2-_n_yolo_ind);
				outputTensor.grid_w = (m_InputW / 32) * pow(2, 2-_n_yolo_ind);
			}

This code runs for yolov4-tiny but it is wrong! In TRT8 branch type "yolov4-tiny" was removed. It need to return it.

@enazoe what do you think about it?

@enazoe
Copy link
Owner

enazoe commented Nov 23, 2021

@Nuzhny007 @auto-amc sry, some trt api is removed in TRT8 ,so tiny model is not support now .

@Nuzhny007
Copy link

@enazoe actually no. I'm merged your master and TRT8 branches and yolov4-tiny works both on TensorRT 7 and TensorRT 8: https://github.com/Smorodov/Multitarget-tracker/tree/master/src/Detector/tensorrt_yolo
If you want then I create a big pull request with all my changes. Or you can download sources from Multitarget-tracker repository and take some changes only for yolov4-tiny.

@auto-amc
Copy link
Author

auto-amc commented Nov 23, 2021 via email

@Nuzhny007
Copy link

Nuzhny007 commented Nov 23, 2021

Here: #158
I don't sure that all changes will be useful for your repository. I'm using this module in my tracker on Windows/Ubuntu x86 and Jetson NX.

@auto-amc I don't know about yolov4-tiny_contrastive. Never used it

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

No branches or pull requests

3 participants