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] 미션 API 마이그레이션 #83

Merged
merged 6 commits into from
Nov 28, 2023
Merged

Conversation

05AM
Copy link
Member

@05AM 05AM commented Nov 27, 2023

☀️ 작업 사항

  • 객체 인식 미션 API 구현: Microsoft azure object detection
  • 표정 인식 미션 API 구현: Google cloud vision - image analysis - face recognition

☀️ 관련 이슈

related: #80

☀️ 참고 사항

  • data.sql의 미션 객체 데이터에 변경사항이 있으나 dev와 local db 모두 수정했습니다.
  • 개체 탐지의 객체 목록이 문서에 명시되어 있지 않아 db에 반영이 되지 않았습니다. 따라서 정확한 결과가 반환되지 않을 수 있습니다. 추후에 그룹화하며 리팩토링 예정입니다.
    ex) 강아지 사진 -> 강아지 품종이 결과로 나옴 -> 미션 실패

05AM added 6 commits November 26, 2023 22:48
- naver에서 microsoft azure image analysis api로 이전
naver -> Google cloud vision api로 이전
- joy, anger, sorrow, surprise 4개로 수정
## ☀️ 작업 사항
- 표정 인식 미션 객체 수정
- 표정 인식 ENUM 타입으로 파라미터 검사

## ☀️ 관련 이슈
related: #80
@05AM 05AM added the FEATURE 새로운 기능 label Nov 27, 2023
@05AM 05AM requested a review from hye-on November 27, 2023 14:01
@05AM 05AM self-assigned this Nov 27, 2023
@Bean
GoogleCredentials googleCredentials() {
try (InputStream serviceAccount = new ClassPathResource(resourceName).getInputStream()) {
GoogleCredentials GoogleCredentials() {
Copy link
Contributor

Choose a reason for hiding this comment

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

대문자로 시작하게 수정한 이유가 궁금해요!

Copy link
Member Author

Choose a reason for hiding this comment

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

착오가 있었던것 같습니다! 수정하겠습니다

GoogleCredentials googleCredentials() {
try (InputStream serviceAccount = new ClassPathResource(resourceName).getInputStream()) {
GoogleCredentials GoogleCredentials() {
try (InputStream serviceAccount = new ClassPathResource(credentials).getInputStream()) {
Copy link
Contributor

Choose a reason for hiding this comment

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

이름 수정한 이유가 궁금해요!
26번째 줄의 credentials하고 같은 이름으로 수정할려고 한 것일까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

네 resourcename이 애매한 변수명이라고 생각하여 수정했습니다

Copy link
Contributor

@hye-on hye-on left a comment

Choose a reason for hiding this comment

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

lgtm~!
고생하셨습니다!

@hye-on hye-on merged commit d39bca1 into main Nov 28, 2023
1 check passed
@05AM 05AM deleted the feature/mission-api-migration branch February 6, 2024 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FEATURE 새로운 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants