Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Frame crash issue resolved #140

Merged
merged 1 commit into from
Aug 29, 2021
Merged

Conversation

ritikchinu
Copy link
Contributor

Issue Number

ISSUE #135

Describe the changes you've made

The issue involved empty frame being captured by caffe based face detection algorithm when we approach close to the camera lens. I have added necessary conditional statement before pre-processing the facial frame to avoid the framework from crashing while executing pre-processing methods in empty frame.

####Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA

####Additional context (OPTIONAL)

####Test plan (OPTIONAL)

Checklist

  • My code follows the code style of this project.
  • I have performed a self-review of my own code.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have commented my code, particularly in hard-to-understand areas.
  • My changes generate no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • The title of my pull request is a short description of the requested changes.

The issue involved empty frame being captured by caffe based face detection algorithm when we approach close to the camera lens. I have added necessary conditional statement before pre-processing the facial frame to avoid the framework from crashing while executing pre-processing methods in empty frame.
@chandrikadeb7 chandrikadeb7 merged commit ba6a915 into chandrikadeb7:master Aug 29, 2021
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.

None yet

2 participants