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

[機能][改善]初期化処理の軽量化 #12

Closed
xpadev-net opened this issue May 31, 2022 · 8 comments
Closed

[機能][改善]初期化処理の軽量化 #12

xpadev-net opened this issue May 31, 2022 · 8 comments
Assignees
@xpadev-net
Labels
[ブラウザ]Chromium Chrome/Edge enhancement New feature or request
Milestone
ver 0.3.0

Comments

@xpadev-net
Copy link
Owner

現状すべてのコメントをmeasureTextしているが、
中を流れるコメントは当たり判定的に消費する範囲は必ず一定で、
高さもDRなどが存在しないためある程度絞れる

そこで、nakaのmeasureTextはgetImage時に行い、初期化時はコマンドを処理して高さ計算をするだけに変更する
※ueやshitaのコメントは今まで通りmeasureTextを行う必要がある
@xpadev-net xpadev-net added the enhancement New feature or request label May 31, 2022
@xpadev-net xpadev-net self-assigned this May 31, 2022
@xpadev-net xpadev-net added this to the ver 0.3.0 milestone Jun 2, 2022
@xpadev-net
Copy link
Owner Author

xpadev-net commented Jun 9, 2022 •
edited
Loading

当たり判定計算時に横幅がわからないといつまで判定が残るのかわからないためこの方法での軽量化はできないが
下の画像のようにgetCommentPos、getCommentSize、sortCommentがそれぞれ3割くらいなので残りの2/3をどうにか軽量化したい
なんならgetCommentPosのほうがgetCommentSizeより重い
image

@xpadev-net
Copy link
Owner Author

xpadev-net commented Jun 9, 2022 •
edited
Loading

image
とりあえずsortCommentを軽量化
ver 0.2.11に適用済み

@xpadev-net
Copy link
Owner Author

image
image
chromiumにて異常にgetCommentSizeが遅い
初期化所要時間：brave 60s+, firefox 10s

@xpadev-net
Copy link
Owner Author

cpuが足を引っ張っているだけの可能性あり
brave@m1
image

@xpadev-net
Copy link
Owner Author

firefox@m1
image
どの道chromium系がfirefoxより遅いのは確かな模様

@xpadev-net xpadev-net moved this to In Progress in niconicomments Aug 13, 2022
@xpadev-net
Copy link
Owner Author

spanをbodyにinsertしてそれに対してfont-sizeを変更し、getBoundingClientRectを実行すれば横幅はmeasureTextと同等のものが得られるかも?
(高さは合わなかった)

@xpadev-net
Copy link
Owner Author

xpadev-net commented Aug 29, 2022 •
edited
Loading

image

むしろ悪くなったorz
しかも微妙に数値合わないし

@xpadev-net
Copy link
Owner Author

xpadev-net commented Sep 6, 2022 •
edited
Loading

image

f8543a4 再帰呼び出しを軽減し、getCommentSizeを高速化
chrome@winで大体40秒くらい短縮

Repository owner moved this from In Progress to Done in niconicomments Sep 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xpadev-net xpadev-net

Labels
[ブラウザ]Chromium Chrome/Edge enhancement New feature or request
Projects
niconicomments
Archived in project
Milestone
ver 0.3.0
Development

No branches or pull requests
1 participant
@xpadev-net