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

Potential memory leak #22

Closed
oyarzun opened this issue Aug 30, 2022 · 6 comments · Fixed by #26
Closed

Potential memory leak #22

oyarzun opened this issue Aug 30, 2022 · 6 comments · Fixed by #26

Comments

@oyarzun
Copy link

oyarzun commented Aug 30, 2022

While testing the webcam example on a raspberry pi, top shows that the memory continues to grow.

Is anyone else seeing this behavior?

@vladimirvivien
Copy link
Owner

Thanks for bringing this to my attention.
Which RPi (3 or 4) ?
Also, how many connections were opened?

@oyarzun
Copy link
Author

oyarzun commented Aug 31, 2022

It was on a RPi 3 B+. I initially tried it with just one connection then added another 2, but the memory increases at about the same rate.

@vladimirvivien
Copy link
Owner

@oyarzun I was able to replicate the issue while running the webcam example. I am going to spend sometimes this weekend to investigate. Thanks again for checking the project out and take time to point this out.

@vladimirvivien
Copy link
Owner

@oyarzun please check out the fix.

@vladimirvivien
Copy link
Owner

Update: I ran a camera feed for about 25 consecutive hours,

  • mem consumption remained stable at 1 %
  • CPU utilization constant at about 3 %
  • Pushing 30 fps HD to browser

@oyarzun
Copy link
Author

oyarzun commented Sep 7, 2022

@vladimirvivien I also confirmed that there is no longer a memory leak. Thanks for fixing the leak and the library in general.

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 a pull request may close this issue.

2 participants