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

[#92] 전체, 연간, 월간 행복 종합 리포트 API 구축 #93

Merged
merged 7 commits into from
May 26, 2024

Conversation

yel-m
Copy link
Member

@yel-m yel-m commented May 25, 2024

#️⃣ 연관된 이슈

Resolves #92

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요 (이미지 첨부 가능)

image

현재 AWS 계정 이슈로 Github Action에서 계속 Fail이 뜰텐데 무시하셔두 됩니다!

  • 가장 행복도가 높았던 활동, 시간대, 장소를 조회하는 API를 구축했습니다.
  • 시간대 나누기
    • 시간대를 가려내기 위한 TimePeriodenum을 생성했습니다. @JsonCreator@JsonValue를 통해 역직렬화와 직렬화를 위한 메서드를 생성하고, 인자로 hour를 받아 아침, 점심, 저녁, 밤 중 어느 시간대에 해당하는지 가려내는 of 메서드를 생성했습니다.
    • 기준
      • 새벽: 0시 ~ 5시
      • 아침: 5시 ~ 9시
      • 낮: 9시 ~ 17시
      • 저녁: 17시 ~ 21시
      • 밤: 21시 ~ 24시
  • 가장 행복한 활동을 가려내는 기준
    • 예를 들어, 가장 행복한 활동을 찾는다고 하면, 활동 별로 그룹화를 하여, 행복도 평균이 가장 높았던 활동들을 가려냅니다.
    • 그 중 빈도가 가장 높은 활동을 가려냅니다.
    • 만약 빈도 또한 같다면, 그 중 랜덤으로 선택하여 보내줍니다.

예시

  • /api/report/all/summary로 GET 요청을 하면 다음과 같은 응답을 받을 수 있습니다. 데이터는 로컬 DB에 있는 더미 데이터를 사용하여 RDS DB 데이터와 다를 수 있습니다!
  • 전체, 연간, 월간 모두 형식이 똑같으므로 전체 예시만 첨부하겠습니다!
{
  "success": true,
  "code": 0,
  "message": "행복 종합 리포트(전체)를 성공적으로 조회했습니다.",
  "data": {
    "timePeriod": "저녁",
    "location": "서울특별시 마포동",
    "activity": "카페 가기"
  }
}
  • 연간, 월간 리포트도 다음과 같은 형식으로 데이터를 조회할 수 있습니다.
  • 시간대와 활동, 장소를 바꿔보며 여러 번 테스트를 진행하였을 때 분석을 꽤 잘 합니다만..... 왜.. 잘되지..? 싶긴 했습니다

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

  • 사소한 것도 괜찮으니 편하게 리뷰 남겨주세요!

✅ Check List

  • PR 제목을 커밋 규칙에 맞게 작성했는가?
  • PR에 해당되는 Issue를 연결했는가?
  • 적절한 라벨을 설정했는가?
  • 작업한 사람을 모두 Assign했는가?

@yel-m yel-m requested a review from KkomSang May 25, 2024 15:57
@yel-m yel-m self-assigned this May 25, 2024
@KkomSang
Copy link
Collaborator

오마이갓,,,,,,, 당신은 ai입니까?? 넘 LGTM입니다!!!!👍 수고하셨습니😍😍😍😍😍😍

@yel-m yel-m changed the title [#92] 행복 종합 리포트 (전체 / 연간 / 월간) API 구축 [#92] 전체, 연간, 월간 행복 종합 리포트 API 구축 May 26, 2024
@KkomSang KkomSang added the feat 새로운 기능을 추가합니다 label May 26, 2024
@KkomSang KkomSang merged commit d71f917 into develop May 26, 2024
1 check failed
@KkomSang KkomSang deleted the feat/#92 branch May 26, 2024 02:43
@yel-m
Copy link
Member Author

yel-m commented May 26, 2024

실제로,,, ai가 맞을지도? (소곤

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새로운 기능을 추가합니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants