[VIEWER-167] Heatmap Viewer 스펙 변경

LTakhyunKim commented 4 weeks ago

📝 Description

Heatmap 표현 방식이 변경됨에 따라 Viewer 내 Heatmap 로직도 업데이트합니다.

업데이트 이미지

확인 방법

npm install script 를 실행해서 패키지를 설치합니다.
npm run start script 를 실행해서 INSIGHT Viewer 문서를 확인합니다.
문서 좌측 navigation sidebar 에서 UseOveralayContext Tab 에서 변경된 Heatmap 을 확인합니다.

✔️ PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Chore (non-breaking change which does not affect codebase; Build related changes, version modification, etc.)
[ ] CI related changes
[ ] Test Case
[ ] Performance optimization
[ ] Site/documentation update
[ ] Other... Please describe:

🎯 Current behavior

구버전 Heatmap Viewer spec 을 사용합니다.

Issue Number: N/A

🚀 New behavior

최신 버전 Heatmap Viewer spec 을 사용합니다.

💣 Is this a breaking change?

[ ] Yes
[x] No

github-actions[bot] commented 4 weeks ago

preview: https://insight-viewer--pr-422.f.lunit.io/

github-actions[bot] commented 4 weeks ago

Coverage report

St.^{:grey_question:}	Category	Percentage	Covered / Total
🟢	Statements	93.02%	80/86
🟢	Branches	94.55%	52/55
🟡	Functions	77.78%	21/27
🟢	Lines	92.31%	72/78

Test suite run success

66 tests passing in 9 suites.

Report generated by 🧪jest coverage report action from c5634c79353dd881e93eddca504b763c84e0ff08

deminoth commented 3 weeks ago

앗 그러고보니 이 PR을 v7에 머지한다면 기존 히트맵이 Legacy가 되면 breaking change에 해당하는 문제가 있네요

LTakhyunKim commented 3 weeks ago

앗 그러고보니 이 PR을 v7에 머지한다면 기존 히트맵이 Legacy가 되면 breaking change에 해당하는 문제가 있네요

중요한 부분 짚어주셔서 감사합니다 🙏 그럼 오히려 반대로 기존 히트맵은 그대로 두고, LastestHeatmap 과 같이 네이밍 해서 추가하는게 좋을까요? v7 을 오래 유지할 것 같진 않아서요.

deminoth commented 3 weeks ago

Lastest는 그때그때 상대적인 개념이니 더 명확한 네이밍이면 좋겠습니다(CXR4Heatmap 이라거나)

LTakhyunKim commented 3 weeks ago

Lastest는 그때그때 상대적인 개념이니 더 명확한 네이밍이면 좋겠습니다(CXR4Heatmap 이라거나)

넵, 명확한 네이밍 고려해서 수정하겠습니다. 🙏

LTakhyunKim commented 3 weeks ago

Lastest는 그때그때 상대적인 개념이니 더 명확한 네이밍이면 좋겠습니다(CXR4Heatmap 이라거나)

CXR4Heatmap 외에 좋은 네이밍이 없어보여 CXR4Heatmap 로 결정했습니다. 8c8496b LegacyHeatmapViewer 는 HeatmapViewer 로 롤백했습니다. edf7dbf

LTakhyunKim commented 3 weeks ago

현재 Github Actions 서비스 자체에 문제가 생겨, 계속 Queued 상태입니다. 리뷰 시 참고 부탁드립니다. https://www.githubstatus.com/ cc. @deminoth

GitHub Status
Welcome to GitHub's home for real-time and historical data on system performance.

lunit-io / insight-viewer