Closed grhaonan closed 9 months ago
Hi, @grhaonan 👋🏻! Great initial version. Let's do a few additional things before we merge:
Change results = model(frame)[0]
into results = model(frame, verbose=False)[0]
. That will prevent printing YOLOv8 logs. There are two places you'd need to change it.
Remove this section:
⚠️ YOLOv8 is still under heavy development. Breaking changes are being introduced almost weekly. We strive to make our YOLOv8 notebooks work with the latest version of the library. Last tests took place on 27.08.2023 with version YOLOv8.0.163.
If you notice that our notebook behaves incorrectly - especially if you experience errors that prevent you from going through the tutorial - don't hesitate! Let us know and open an issue on the Roboflow Notebooks repository.
Could we remove tqdm
progress bar from your code or present an alternative implementation that would not require the usage of global
?
Remove this section:
⚠️ YOLOv8 is still under heavy development. Breaking changes are being introduced almost weekly. We strive to make our YOLOv8 notebooks work with the latest version of the library. Last tests took place on 27.08.2023 with version YOLOv8.0.163.
If you notice that our notebook behaves incorrectly - especially if you experience errors that prevent you from going through the tutorial - don't hesitate! Let us know and open an issue on the Roboflow Notebooks repository.
Remove this section:
⚠️ Disclaimer
YOLOv8 is still under heavy development. Breaking changes are being introduced almost weekly. We strive to make our YOLOv8 notebooks work with the latest version of the library. Last tests took place on 23.01.2023 with version YOLOv8.0.17.
If you notice that our notebook behaves incorrectly - especially if you experience errors that prevent you from going through the tutorial - don't hesitate! Let us know and open an issue on the Roboflow Notebooks repository.
And replace it with:
This notebook uses legacy versions of ByteTrack and Supervision. To be up to date, use our revamped notebook.
@SkalskiP thanks for your review and please see the latest commit included.
Good day @SkalskiP , any update on this? thanks
Good day @SkalskiP , any update on this? thanks
This worked perfectly for me on my M2 Macbook running locally, thanks for the refactor!
@iturner72 It's awesome to hear that! 🔥
@grhaonan, is there a chance you could accept CLA? I'd love to merge this PR. 🔥
@SkalskiP All accepted and thanks for your testing @iturner72
Looks great! Merging! Thank you @grhaonan! 🙏🏻
Thank you all
new version of how-to-track-and-count-vehicles-with-yolov8.ipynb to use the latest Supervision
Description
Please include a summary of the change and which issue is fixed or implemented. Please also include relevant motivation and context (e.g. links, docs, tickets etc.).
List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.
How has this change been tested, please provide a testcase or example of how you tested the change?
I have tested it in colab
Any specific deployment considerations
For example, documentation changes, usability, usage/costs, secrets, etc.
Docs