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

[Feat] Update Face Recognition & Save timeline #54

Merged
merged 1 commit into from
Jun 5, 2022
Merged

Conversation

sooya233
Copy link
Contributor

@sooya233 sooya233 commented Jun 4, 2022

기능 설명

  • 향상된 Face Recognition 모델을 적용하였다.
  • 이전에는 timeline에 대한 리스트를 client에 직접 제공하였지만, 이를 따로 저장하여 response 및 request가 길어졌을 때의
    속도 하락을 방지하였다.
  • 그러나, 현재는 큰 속도 향상이 이루어지지 않았는데, 이는 laugh detection을 동시에 진행함으로 인하여 발생하는 문제로 보인다.
  • 따라서, laugh detection을 완전히 병렬적으로 response를 client에게 전송할 수 있는 방법을 찾아야 할 것 같다.

Key Changes

  • modified: serving/backend/app/api/face.py
  • modified: serving/backend/app/api/highlight.py
  • modified: serving/backend/app/ml/face_functions.py
  • modified: serving/backend/app/ml/face_recognizer/face_recog.py

Related Issue

close #53


- 향상된 Face Recognition 모델을 적용하였다.
- 이전에는 timeline에 대한 리스트를 client에 직접 제공하였지만, 이를
따로 저장하여 response 및 request가 길어졌을 때의 속도 하락을
방지하였다.
- 그러나, 현재는 큰 속도 향상이 이루어지지 않았는데, 이는 laugh
detection을 동시에 진행함으로 인하여 발생하는 문제로 보인다.
- 따라서, laugh detection을 완전히 병렬적으로 response를 client에게
전송할 수 있는 방법을 찾아야 할 것 같다.
issue #53
@sooya233 sooya233 added Serving Serving 관련 이슈 BE Serving 중 Backend 관련 이슈 labels Jun 4, 2022
@sooya233 sooya233 requested a review from seungriyou June 4, 2022 11:32
@sooya233 sooya233 self-assigned this Jun 4, 2022
Copy link
Contributor

@seungriyou seungriyou left a comment

Choose a reason for hiding this comment

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

LGTM

@seungriyou seungriyou merged commit e593c9d into develop Jun 5, 2022
@seungriyou seungriyou deleted the feat/53 branch June 5, 2022 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE Serving 중 Backend 관련 이슈 Serving Serving 관련 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants