Piotr Skalski

@rolson24 1. I think we should solve this issue first and then merge the `minimum_consecutive_frames` PR. Let's keep those two separate. 2. I'd like to ask you to thoroughly test...

Hi @rolson24 👋🏻 I left a comment regarding: ```python invalid_tracks = [ i for i in range(len(detections.tracker_id)) if detections.tracker_id[i] != -1 ] return detections[invalid_tracks] ```

@rolson24 I'm merging this PR. As for ByteTrack unit tests, I would love this! Feel free to open the next PR with tests.

Change implemented via https://github.com/roboflow/supervision/pull/1046. We are closing the issue.

Hi @Griffin-Sullivan~ 👋🏻 Yes `mask_to_xyxy` is what you need. Looks like your problem was mostly related to `np.array` `dtype`. I shared more details in the comment under your PR.

This issue was solved via https://github.com/roboflow/supervision/pull/1054. I'm closing this issue.

@LinasKo are those potential changes reflected in comments under this PR: https://github.com/roboflow/supervision/pull/1046, or is it more than this?

Hi @zhmiao 👋🏻 In fact, Supervision does not change the channel order. By default, Supervision uses OpenCV as its image/video processing engine, and BGR is the default color order for...

@onuralpszr, thanks a lot for creating this issue 🙏🏻 I really appreciate that you're so helpful.

Hi @onuralpszr are you still thinking about finishing this work, or can I make it unassigned?