feat: 🚀 Add default color to PolygonZoneAnnotator, drawing functions

[DivyaVijay1234]

Description

Default Colors Added: Default colors were added to utility functions such as drawLine, drawRectangle, drawPolygon, and PolygonZoneAnnotator.
Documentation: Documentation was added to explain the changes and usage of the new default colors.

Fix: #1587

Type of change

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

[CLAassistant]

All committers have signed the CLA.

[github-actions]

Preview

Preview and run these notebook edits with Google Colab:

• docs/notebooks/small-object-detection-with-sahi.ipynb

Rendered notebook diffs available on ReviewNB.com. If commits are added to the pull request, synchronize your local branch: git pull origin develop

[onuralpszr]

Please use our color scheme in like "sv.Color.WHITE" for all them. You can find them in here

supervision/supervision/draw/color.py

Line 231 in 337b0e4

def WHITE(cls) -> Color:

[onuralpszr]

@DivyaVijay1234 please accept CLA and please check my review and after that we can move on thank you.

[DivyaVijay1234]

@onuralpszr Which of those colours would you like me to use as the default colour?

[onuralpszr]

@onuralpszr Which of those colours would you like me to use as the default colour?

Change all of "your" changes to default color like "white" I wrote in comment

[LinasKo]

Hi @DivyaVijay1234 👋

Thank you for the contribution!

Which of those colours would you like me to use as the default colour?

The default color we prefer is Color.ROBOFLOW. I see you set the values for it, but that's in fact not necessary. You can simply replace Color(r, g, b) with Color.ROBOFLOW.

[DivyaVijay1234]

@onuralpszr @LinasKo
Thank you for guidance. Please confirm if I need to make any further changes or if the issue has now been solved.

[onuralpszr]

@onuralpszr @LinasKo Thank you for guidance. Please confirm if I need to make any further changes or if the issue has now been solved.

push code please :)

[DivyaVijay1234]

Please check if the changes I made are now visible. Thank you.

[onuralpszr]

@DivyaVijay1234 I made some small documentation additions and also add white color to other annotator you missed with documentations, thank you for your contribution I am merging in this and close the issue as complete as well.

cc @LinasKo

[onuralpszr]

I made some fixes on PR title and comment part to be more descriptive so that everyone else can understand what is going on in here as well.

[onuralpszr]

LGTM !

[DivyaVijay1234]

@onuralpszr Thank you!

[LinasKo]

@onuralpszr , the documentation updates are helpful, but I'm confused why PolygonAnnotator got white as the default color. Does this also pick it for text boxes? Otherwise, the line should be roboflow-colored.

[onuralpszr]

@onuralpszr , the documentation updates are helpful, but I'm confused why PolygonAnnotator got white as the default color. Does this also pick it for text boxes? Otherwise, the line should be roboflow-colored.

The reason I used white because I wanted to follow LineZoneAnnotator also using white lines

https://supervision.roboflow.com/latest/detection/tools/line_zone/#supervision.detection.line_zone.LineZoneAnnotator-functions

__init__(thickness=2, color=Color.WHITE, text_thickness=2, text_color=Color.BLACK, text_scale=0.5, text_offset=1.5, text_padding=10, custom_in_text=None, custom_out_text=None, display_in_count=True, display_out_count=True, display_text_box=True, text_orient_to_line=False, text_centered=True) 

[LinasKo]

@onuralpszr , that makes sense! I did not notice that. Thank you for the guidance and PR resolution!

[onuralpszr]

@onuralpszr , that makes sense! I did not notice that. Thank you for the guidance and PR resolution!

Pleasures 🙏 ☺️