merging detections prints "hm"

[matthew-fall-apl]

Search before asking

• I have searched the Supervision issues and found no similar bug report.

Bug

When using Detections.merge() on detections that have information in their metadata, "hm" gets printed.

Environment

• supervision >= 0.25.1
• python == 3.10.9
• OS: Windows 11 or Linux Ubuntu 22.04

Minimal Reproducible Example

from supervision import Detections
import numpy as np

detections1 = Detections(xyxy=np.array([[10,10,50,50]]),
                         metadata={'frame':1})

detections2 = Detections(xyxy=np.array([[20,20,70,70]]),
                         metadata={'frame':1})

merged_detections = Detections.merge([detections1, detections2])

Additional

No response

Are you willing to submit a PR?

• Yes I'd like to help by submitting a PR!

[kshitijaucharmal]

Weird bug, reproduced successfully on 0.26.1.

Observations

1. Does not show up when there's no metadata
2. Prints as many times as there are elements in the metadata dict

[kshitijaucharmal]

Have put up a PR, was just a stray print("hm") in the code. You can close the issue as soon as its merged

[matthew-fall-apl]

Thanks! I also noticed observation 1 (which is probably why this did not get caught in development). In my case, I only used one metadata key, so I never noticed observation 2.

I subscribed to your PR, so I'll keep an eye on its status.

[SkalskiP]

I just merged the PR fixing this issue. Thanks a lot @kshitijaucharmal and @matthew-fall-apl.

[matthew-fall-apl]

thanks!