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

few quotations on gpu version #6

Open
yaira24 opened this issue Apr 8, 2019 · 2 comments
Open

few quotations on gpu version #6

yaira24 opened this issue Apr 8, 2019 · 2 comments

Comments

@yaira24
Copy link

yaira24 commented Apr 8, 2019

First of all good job on MTCNN GPU version!

why the image size need to configure up front in the constructor?
is there a resin why you comment the mtcnn::~mtcnn()?

@PKUZHOU
Copy link
Owner

PKUZHOU commented Apr 24, 2019

Q1: TensorRT doesn't support dynamic input size, so the image size should be configured previously.
Q2: There is something wrong with the destructor

@yaira24
Copy link
Author

yaira24 commented Jun 30, 2019

thanks for the answer
about the destructor, I change the code there and I don't have any issue
mtcnn::~mtcnn(){

delete[] pnet_engine;
delete rnet_engine;
delete onet_engine;
for (int i = 0;i<scales_.size();i++)
{
    delete(simpleFace_[i]);
}
delete[] simpleFace_;
simpleFace_ = NULL;
delete refineNet;
delete outNet;
free(boxes_data);
CHECK(cudaFree(gpu_boxes_data));

}

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

2 participants