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

Fix memory leak in ByteTracker #249

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix memory leak in ByteTracker #249

wants to merge 2 commits into from

Conversation

sr99622
Copy link

@sr99622 sr99622 commented Jul 27, 2022

Problem with ByteTracker during long running process. Virtual memory is consumed in same manner as memory leak. This is due to the self.removed_stracks list member variable growing without bound. The issue may be fixed by removing stale tracks from the self.removed_stracks list. A limit is placed such that tracks older than 10 * self.max_time_lost are removed from the self.removed_stracks list. This change had no discernible effect on the algorithm.

Problem with ByteTracker during long running process.  Virtual memory is consumed in same manner as memory leak.  This is due to the self.removed_stracks list member variable growing without bound.  The issue may be fixed by removing stale tracks from the self.removed_stracks list.  A limit is placed such that tracks older that 10 * self,max_time_lost are removed from the self.removed_stracks list.  This change had no discernible effect on the algorithm.
@sr99622 sr99622 changed the title Update byte_tracker.py Fix memory leak in ByteTracker Jul 27, 2022
pytorch uint8 to bool for deprecation
@Vladimir-125
Copy link

Thanks!

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