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

[FEATURE]フロントエンドのパフォーマンス改善 #8

Open
nasuka opened this issue Dec 17, 2024 · 2 comments
Open

[FEATURE]フロントエンドのパフォーマンス改善 #8

nasuka opened this issue Dec 17, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@nasuka
Copy link
Collaborator

nasuka commented Dec 17, 2024

背景

  • 全画面地図のデータ点が増加した場合に動作が重くなる
    • 特にスマートフォン等のスペックの低い端末では、クラッシュすることがある

提案内容

  • 案としては以下が想定されるが、アプローチを検討するところから実施する
    • 解決案
      • 全画面地図をSVG -> canvas化することでパフォーマンスを向上させる

その他

恐らく全画面地図の描画動作が重くなっていると思われる
全画面地図の描画はデバイスによってファイルが分かれており、それぞれ以下のファイルで実装されている。
現状ではスマホ側がボトルネックになっているが、PC側でもデータ点の増加に応じて動作が重くなる事象が確認されており、できれば両方で動作を改善したい。(よりクリティカルなのはスマホ側ではある)

  • スマホ: MobileMap.tsx
  • PC: DesktopMap.tsx
@nasuka nasuka added the enhancement New feature or request label Dec 17, 2024
@nasuka nasuka changed the title [FEATURE]フロントエンドの高速化 [FEATURE]フロントエンドのパフォーマンス改善 Dec 17, 2024
@nishio
Copy link
Collaborator

nishio commented Dec 18, 2024

解決案
全画面地図をSVG -> canvas化することでパフォーマンスを向上させる

30個のクラスタの見出し部分に出ている散布図を消すだけで3GBのメモリ消費が100MBになると思う。(GTTは今そうやってるんだっけ)
canvas化は「操作可能な状態を保つため」だと思うが、その要件を捨てると画像にする選択肢もありえる。
クラスタの見出し部分は操作できなくてもいいかなという気がする。
特にスマホで見る人は小さい画面で細かい操作をしたいと思っているのかどうか微妙。

@nishio
Copy link
Collaborator

nishio commented Dec 23, 2024

indexの下の方のクラスタ詳細ビューをsetTimeoutとかで分離したら最初の散布図が出るまでの時間が1/31になったりしないかな

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
None yet
Development

No branches or pull requests

3 participants