Skip to content

fix: prevent multiple image windows for multi-face images#1676

Open
gauris1104 wants to merge 1 commit into
ageitgey:masterfrom
gauris1104:fix/1666-image-display-loop
Open

fix: prevent multiple image windows for multi-face images#1676
gauris1104 wants to merge 1 commit into
ageitgey:masterfrom
gauris1104:fix/1666-image-display-loop

Conversation

@gauris1104

Copy link
Copy Markdown

Summary

This PR fixes the issue where pil_image.show() was being called inside the face-processing loop.

Changes

  • Moved pil_image.show() outside the for face_landmarks in face_landmarks_list loop.
  • Ensured the image is displayed only once after all detected faces have been processed.

Result

  • Prevents multiple image viewer windows from opening when an image contains multiple faces.
  • Improves user experience without changing existing functionality.

Fixes #1666

@gauris1104

Copy link
Copy Markdown
Author

I've submitted a PR for this issue: #1666

The fix moves pil_image.show() outside the face-processing loop so the image is displayed only once after all faces have been processed.

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.

Logical Error: Image display (.show()) placed inside loop, causing multiple windows for multi-face images

1 participant