fix: allow negative offsets in move_masks function (#1715)

[Anirudh2112]

Description

This PR fixes the restriction on negative offsets in the move_masks function. Previously, the function would raise a ValueError for negative offset values, but similar functions like move_detections and move_oriented_boxes support negative offsets. This change makes the behavior consistent across all movement functions.

Fixes #1715

Type of change

• Bug fix (non-breaking change which fixes an issue)

How has this change been tested

Added comprehensive unit tests covering:

• Positive offset (existing behavior)
• Negative offset (new behavior)
• Zero offset
• Partial out of bounds (positive)
• Partial out of bounds (negative)
• Multiple masks with different patterns

Example test case:

masks = np.array([[[True, True], [True, True]]])
offset = np.array([-1, -1])
resolution_wh = (3, 3)
result = move_masks(masks=masks, offset=offset, resolution_wh=resolution_wh)

[CLAassistant]

All committers have signed the CLA.

[SkalskiP]

Hi @Anirudh2112; thank you so much for your interest in Supervision and this specific task. I took the liberty of making a few changes to the solution you designed because I noticed it wasn’t working correctly with mixed offsets, for example, [-2, 2].

[Anirudh2112]

@SkalskiP, thank you for reviewing and improving the solution! This was a great learning experience for me in terms of handling edge cases and comprehensive testing. I'll be sure to consider such mixed cases in future contributions.

[SkalskiP]

@Anirudh2112 great to hear that; I can't wait to see your future contributions!