Skip to content

[fix] Solve the memory leak problem.#8

Open
somone23412 wants to merge 1 commit into
deepinsight:masterfrom
somone23412:patch-1
Open

[fix] Solve the memory leak problem.#8
somone23412 wants to merge 1 commit into
deepinsight:masterfrom
somone23412:patch-1

Conversation

@somone23412
Copy link
Copy Markdown

Solve the serious memory leak problem caused by unreleased PNET while Detect() is called many times.

Hello, I really appreciate your work.

When I used this code, I found that frequent calls to the Detect() method (I only instantiated one MTCNN object) resulted in an abnormal increase in memory.

Finally i found that in the mxnet_mtcnn.hpp, you have said "PredictorHandle PNet_; //PNet_ will create and destroyed frequently", but there is no release of PNet in the source code.

I see that you have written the destroy function for Pnet, so I wonder if you forgot to add it. When I tried to add it, the memory leak problem never happened again.

Solve the serious memory leak problem caused by unreleased PNET while Detect() is called many times.
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.

1 participant