Onuralp SEZER
Onuralp SEZER
> > @hellcp or @cboltz, I'm not knowledgeable enough to merge this. Can you take a look at it please. > > yes Can you please write more descriptive title...
Supersede by https://github.com/ultralytics/ultralytics/pull/22763
> We need a Matrix room icon as well for https://matrix.to/#/#cloud:opensuse.org π flying gecko ftw ! :)
@DivyaVijay1234 please accept CLA and please check my review and after that we can move on thank you.
> @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
> @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 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...
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 , 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...
> @onuralpszr , that makes sense! I did not notice that. Thank you for the guidance and PR resolution! Pleasures π βΊοΈ