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.

Sign up for GitHub

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

Jump to bottom

Chart > Legend 영역 통계 > null 처리 #1269

Closed
jhee564 opened this issue Aug 23, 2022 · 1 comment
Closed

Chart > Legend 영역 통계 > null 처리 #1269

jhee564 opened this issue Aug 23, 2022 · 1 comment
Labels
inactive

Comments

@jhee564
Copy link
Collaborator

jhee564 commented Aug 23, 2022

이슈 내용

  • chart 데이터로 null과 undefined 가 섞여 들어올 수 있으며 계산 (합계, 평균, 최소, 최대) 시 null은 0으로 대체되어 계산되며 undefined은 NaN이 됨

개선 방향

  • Null 과 Undefined 값은 제외하고 계산할 필요 있음
jhee564 added a commit that referenced this issue Aug 23, 2022
@jhee564
[#1269] Chart > Legend 영역 통계 > null 처리
6dda85b 
#####################################
- 합계, 최솟값, 최댓값, 평균 계산시 null과 undefined는 제외하도록 함
- 마지막값 계산 시 null과 undefined는 'Null'로 표시되도록함
- 계산된 값이 javascript기준 MAX_SAFE_INTEGER을 초과했거나 MIN_SAFE_INTEGER 미만일 경우 console warning이 출력되도록 함
jhee564 added a commit that referenced this issue Aug 23, 2022
@jhee564
[#1269] Chart > Legend 영역 통계 > null 처리 (#1270)
f17a2d3 
#####################################
- 합계, 최솟값, 최댓값, 평균 계산시 null과 undefined는 제외하도록 함
- 마지막값 계산 시 null과 undefined는 'Null'로 표시되도록함
- 계산된 값이 javascript기준 MAX_SAFE_INTEGER을 초과했거나 MIN_SAFE_INTEGER 미만일 경우 console warning이 출력되도록 함

Co-authored-by: jinhee park <jinhee@ex-em.com>
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 2, 2024

오래된 이슈는 닫고 있어요. 필요하다면 다시 열어주세요. 😄

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
inactive
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jhee564