supervision icon indicating copy to clipboard operation
supervision copied to clipboard
verified publisher icon roboflow

Improve number of iterations over tracks by popping instead of None filling

Open DACUS1995 opened this issue 1 year ago • 4 comments

Description

Suggested an alternative to using None objects as placeholders for missing detection. At each detection update, we remove the oldest detection for a tracker and when that queue is empty simply remove it. This way we don't need to filter and check the presence of None.

Type of change

Please delete options that are not relevant.

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] This change requires a documentation update

DACUS1995 avatar Jan 25 '24 21:01 DACUS1995

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jan 25 '24 21:01 CLAassistant

Hi, @DACUS1995 👋🏻! Thanks for your interest in supervision. Looks like the right move. I will need to test out the code. I can't do it now - I am working on a roadmap for the upcoming months. However, we will remember this PR in the context of supervision-0.19.0.

SkalskiP avatar Jan 25 '24 22:01 SkalskiP

Sorry, closed it by mistake, let me know if there is anything I should change.

DACUS1995 avatar Feb 01 '24 14:02 DACUS1995

Hi @DACUS1995 👋🏻 No worries. I was looking at this PR today. I think this is not an equal solution.

SkalskiP avatar Feb 01 '24 15:02 SkalskiP