Skip to content

Conversation

martinkronberg
Copy link

Added notebook
verified that it runs as expected
created readme

Added notebook
verified that it runs as expected
created readme
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

matched overall style to other notebooks with cell and step descriptions
@martinkronberg
Copy link
Author

@aleksandr-mokrov Hey Aleksander - could you take a look at this PR?

@@ -0,0 +1,670 @@
{
Copy link
Collaborator

@aleksandr-mokrov aleksandr-mokrov Oct 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line #1.    %pip install -q "openvino>=2023.1.0" "ultralytics==8.3.0" opencv-python matplotlib Pillow ipywidgets --extra-index-url https://download.pytorch.org/whl/cpu

Update please openvino version to the latest release: 2025.3


Reply via ReviewNB

@@ -0,0 +1,670 @@
{
Copy link
Collaborator

@aleksandr-mokrov aleksandr-mokrov Oct 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

YOLOv11n-cls -> YOLOv11n-cls to pass the spell-check or add the word cls to .ci\spellcheck\.pyspelling.wordlist.txt


Reply via ReviewNB

@@ -0,0 +1,670 @@
{
Copy link
Collaborator

@aleksandr-mokrov aleksandr-mokrov Oct 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't add please image in the repo. You can add the images to any comment in PR and use that link for downloading.


Reply via ReviewNB

@@ -0,0 +1,670 @@
{
Copy link
Collaborator

@aleksandr-mokrov aleksandr-mokrov Oct 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line #3.    if detect_device.value != "CPU":

Could you try to use intel device for inference for detect and classification models, like it is in yolov11-object-detection (without quantization):

res = det_model(IMAGE_PATH, device=f"intel:{device.value.lower()}")

It should simplify and reduce the number of extra code lines.


Reply via ReviewNB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants