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] Add upload-video and show-people api #14

Merged
merged 4 commits into from
May 27, 2022
Merged

[Feat] Add upload-video and show-people api #14

merged 4 commits into from
May 27, 2022

Conversation

seungriyou
Copy link
Contributor

기능 설명

  • /serving/backend/app/api 내부에는 큰 기능별로 파일을 나누었습니다.
    • laughter.py의 경우, 아직은 작성하지 않았습니다.
  • /serving/backend/app/ml에 face clustering(extracting) 관련 코드가 위치합니다.
    • 실제로 api에서 사용하는 모듈은 extract_face.py 입니다.

Key Changes

  • 기존의 /serving/backend 폴더를 삭제하고 다시 올립니다.

Related Issue


To Reviewers

  • /serving/backend/app/ml 폴더를 어떻게 정리할지 고민입니다.
    • 추후 face recognition, laughter detection 코드도 들어갈 것이기 때문에 체계적으로 정리해야 할 듯 합니다.
  • 개발 중에 사용할 config 파일이 필요합니다. (ip 주소 등)
  • face clustering(extracting) 관련해서 오늘 업데이트 된 점(last_id 문제 해결, clustering 방법 도입, cloth 이용 등)을 아직 반영하지 않았습니다.

첫 번째 backend baseline을 삭제합니다.

issue #11
backend 베이스라인과 /upload-video api를 구현하였습니다.
issue #11
face clustering(extracting)을 수행한 후 그 결과를 반환하는 api를
구현하였습니다.
issue #11
backend 베이스라인과 /upload-video api를 구현하였습니다.
issue #11
@seungriyou seungriyou added Serving Serving 관련 이슈 BE Serving 중 Backend 관련 이슈 labels May 26, 2022
@seungriyou seungriyou requested a review from sooya233 May 26, 2022 14:06
@seungriyou seungriyou self-assigned this May 26, 2022
Copy link
Contributor

@sooya233 sooya233 left a comment

Choose a reason for hiding this comment

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

LGTM

@sooya233 sooya233 merged commit 1b1499e into develop May 27, 2022
@sooya233 sooya233 deleted the feat/11 branch May 27, 2022 00:43
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