angular-ui/ui-grid

Grid is build twice for each data-change when fastWatch is enabled

Open

#6,172 opened on 2017年4月24日

GitHub で見る
 (0 comments) (1 reaction) (0 assignees)JavaScript (5,395 stars) (2,496 forks)batch import
help wantedseverity: performance

説明

Can be observed on the tutorial page: http://ui-grid.info/docs/#/tutorial/404_large_data_sets_and_performance

Due to these lines (https://github.com/angular-ui/ui-grid/blob/19455276ace7eef5c8dab8f719c17d7c4c31ae3a/src/js/core/directives/ui-grid.js#L56-L57) assigning a new array to the data property rebuilds the rows twice for the same change.

This has a major impact on load times of the grid, when complex rowProcessors are registered (treeBase, sort, etc.)

コントリビューターガイド

技術スタック
javascript
領域
frontend
Issue 種別
performance
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
AngularJSJavaScriptui grid architecture
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40
調査方針
Investigate the lines 56-57 in src/js/core/directives/ui grid.js where a new array assignment triggers double rebuild. Understand the fastWatch mechanism and how data changes are propagated. Propose a fix to avoid rebuilding rows twice. Check for any existing tests for this behavior.