Skip to content

Conversation

@kkawabat
Copy link
Contributor

@kkawabat kkawabat commented Aug 10, 2023

The landmarks_demo.py fails with a cryptic error when no faces are detected in the input image. Added some error handling so that this behavior is more transparent.

See issue here and forum discussion here.

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

…etected in the input image. Added some error handling so that it's more transparent.
@asmorkalov asmorkalov self-requested a review August 11, 2023 10:25
@asmorkalov asmorkalov merged commit 9e13469 into opencv:4.x Aug 11, 2023
@asmorkalov asmorkalov mentioned this pull request Sep 11, 2023
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