Fixed TraceAnnotator to respect the postion argument.

[abhishek7kalra]

Description

Added necessary logic in TraceAnnotator so that it now respects the position argument. The update now allows all the predefined positions which when passed to modify the tracing position.

Issue: #420

Type of change

Please delete options that are not relevant.

• New feature (non-breaking change which adds functionality)

How has this change been tested, please provide a testcase or example of how you tested the change?

trace_annotator = sv.TraceAnnotator(position=Position.TOP_LEFT)

trace_topleft.mp4

[abhishek7kalra]

@SkalskiP Please review this pr.

[SkalskiP]

Hi, @abhishek7kalra! 👋🏻 Thanks for your interest in Supervision and solving this issue.

However, the solution for this bug is much simpler. The logic that you made is already part of sv.Detections.get_anchors_coordinates method. That method is called inside Trace class. The problem is that the position argument is not being passed to the Trace constructor.

All you have to do is pass `position here, and it should work.

self.trace = Trace(max_size=trace_length, anchor=position)

[SkalskiP]

Hi, @abhishek7kalra! 👋🏻 Unfortunately, I couldn't wait any longer for you to implement the fixes, so I decided to do it myself. Thanks a lot for your help! 🙏🏻 Merging!

[abhishek7kalra]

Hey @SkalskiP, thank you for going ahead with the fix. I got super busy, so couldn't do the implementation myself. 😅