Devrim
Devrim
@jbitton Thank you for review, I updated the PR with the latest changes.
Hi @zpapakipos, past weekend I did pulled the last changes and rebased already, but I didnt pushed the changes due to the bbox update you are working on. Having said...
Hi @zpapakipos, I've updated with the latest changes, I take a look overall, I guess we're good to go. After your review, let me know if any change further is...
> Once you've published those changes we should be able to land your changes, and then I'll work on a follow-up diff for the bboxes. I'm thinking of doing an...
> Hi @devrimcavusoglu, update for you: I imported this PR & tried to get the augmentations working in our internal version of the augly codebase (which is linked to this...
Hi @sergiev thank you for your contribution, I also had a chance to look at previous related PR. Could you maybe add some test data and extend the tests for...
> @devrimcavusoglu hi, thanks for the reply! ~Sorry but I sincerely do not understand what and where exactly you want me to put in... Could you expand the idea?~ >...
@edugzlez Can you add the commits on this PR https://github.com/edugzlez/sahi/pull/1 and we can merge this PR. Moreover, about removing device placement on `Yolov8DetectionModel`, it's kind of undesired. My proposal is...
Thank you @edugzlez, I mentioned 1 more issue in the comment that changing the device behavior in `Yolov8DetectionModel` class could be undesired. Can you move that device change to `RTDETRDetectionModel`...
@edugzlez Thank you for the amazing contribution. 🎉