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] 차트 Zoom 기능 재활성화 #364

Merged
merged 11 commits into from
Dec 4, 2023
Merged

Conversation

yoopark
Copy link
Contributor

@yoopark yoopark commented Oct 29, 2023

Summary

image

Describe your changes

  • 예전에 toolbar가 미관상 좋지 않아 삭제했었으나, 이제 와서 생각해보니 굳이 제공할 수 있는 기능을 막는 것도 좋지 않은 것 같아 재활성화합니다. 기본 toolbar는 필요없는 기능(png로 저장 등)도 지원해서 사용하기 싫었는데 선택적으로 버튼을 없앨 수 있어서 toolbar를 추가해도 괜찮겠다고 생각하였습니다.
  • 레벨 증가 그래프는 2년의 시간이 정해져 있어, 기존 방식과 동일하게 zoomin & zoomout을 하지 못하도록 설정하였습니다.
  • zoomin & zoomout의 대상이 되는 차트는 LineChart, AreaChart이며, 구체적으로는 다음과 같습니다. (우측 숫자는 last 변경 값입니다)
    • [HomeEval] 일간 평가 횟수 추이 : 30 -> 60
    • [HomeTeam] 일간 팀 제출 횟수 추이 : 30 -> 60
    • [HomeUser] 여행 중인 유저 수 추이 : last 지원하지 않음
    • [HomeUser] 월간 블랙홀 인원 추이 : 12 -> 24
    • [HomeCoalition] 코알리숑 스코어 변동 추이 : last 지원하지 않음
    • [PersonalGeneral] 월간 접속 시간 추이 : 12 -> 24
    • [PersonalEval] 월간 평가 횟수 추이 : 12 -> 24
  • BE에 다음의 사항을 물어봐야 합니다.
    • last를 늘리면 서버 부하에 어떤 지장이 있을지 (가능하다면 일간은 last 365일, 여행 중인 유저 수 추이도 월별 말고 일별로 얻고 싶습니다)
    • 여행 중인 유저 수 추이, 코알리숑 스코어 변동 추이에 last를 지원할 수 있는지
    • 여행 중인 유저 수 추이를 월별 -> 일별로 변경할 수 있는지

Issue number and link

@yoopark yoopark added the enhancement New feature or request label Oct 29, 2023
@yoopark yoopark self-assigned this Oct 29, 2023
@yoopark
Copy link
Contributor Author

yoopark commented Oct 30, 2023

@jpham005 혹시 위 세 가지 질문에 답변해주실 수 있나요?

@jpham005
Copy link
Member

jpham005 commented Oct 30, 2023

last 가 커질수록 부하는 커집니다.
언급된 사항 모두 구현상 어려움은 없으나 적절한 최적화를 해야해서 고민을 좀 해야겠네요.

@yoopark
Copy link
Contributor Author

yoopark commented Oct 30, 2023

last 가 커질수록 부하는 커집니다. 언급된 사항 모두 구현상 어려움은 없으나 적절한 최적화를 해야해서 고민을 좀 해야겠네요.

감사합니다! last 늘리는 것에 대하여 회의 때 얘기 나눠봅시다

@yoopark
Copy link
Contributor Author

yoopark commented Nov 2, 2023

백엔드 성능 개선이 필요하기 때문에 지금 도입할 수 있는 Feature가 아닙니다.
브랜치 삭제는 하지 않고 PR을 닫습니다.

@yoopark yoopark closed this Nov 2, 2023
@yoopark yoopark reopened this Nov 22, 2023
@yoopark yoopark force-pushed the feat/enable-chart-zoom branch from 5c7f95c to db7e36c Compare November 24, 2023 06:58
@yoopark yoopark force-pushed the feat/enable-chart-zoom branch from db7e36c to 7600597 Compare November 24, 2023 07:20
@yoopark
Copy link
Contributor Author

yoopark commented Nov 24, 2023

@yoopark yoopark force-pushed the feat/enable-chart-zoom branch from e041678 to 7600597 Compare December 3, 2023 13:41
@yoopark
Copy link
Contributor Author

yoopark commented Dec 3, 2023

last 결정 사항

  • [HomeEval] 일간 평가 횟수 추이 : 180일
  • [HomeTeam] 일간 팀 제출 횟수 추이 : 180일
  • [HomeUser] 여행 중인 유저 수 추이 : 180일 (그런데, 기존 월별로 받는 것을 유지하고, last만 늘려서 받는게 예쁠 것 같다. 아래 사진 참고.)
  • [HomeUser] 월간 블랙홀 인원 추이 : 42 시작달로부터 last 계산
  • [HomeCoalition] 코알리숑 스코어 변동 추이 : 42 시작달로부터 last 계산
  • [PersonalGeneral] 월간 접속 시간 추이 : 해당 유저 본과정 시작달로부터 last 계산
  • [PersonalEval] 월간 평가 횟수 추이 : 해당 유저 본과정 시작달로부터 last 계산

image

감소세가 선형적이라 굳이 일별로 받을 필요 없을 듯

수정된 AreaChart 캡처

image
image
image
image

@yoopark yoopark requested a review from 42sungwook December 3, 2023 15:33
app/src/Home/dashboard-contents/User/UserRate.tsx Outdated Show resolved Hide resolved
app/src/@shared/constants/date.ts Show resolved Hide resolved
app/src/Landing/components/Introduction.tsx Outdated Show resolved Hide resolved
@yoopark yoopark force-pushed the feat/enable-chart-zoom branch from 8a1a4e1 to b97d226 Compare December 4, 2023 10:44
@42sungwook 42sungwook merged commit 41f8c8d into develop Dec 4, 2023
1 check passed
@42sungwook 42sungwook deleted the feat/enable-chart-zoom branch December 4, 2023 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants