Skip to content

[test-viewer] Dependencies bump - #1368

Merged
saskliutas merged 6 commits into
masterfrom
test-viewer-bump
Jul 4, 2025
Merged

[test-viewer] Dependencies bump#1368
saskliutas merged 6 commits into
masterfrom
test-viewer-bump

Conversation

@saskliutas

Copy link
Copy Markdown
Member
  • Bumped test-viewer dependencies.
  • Updated eslint config.
  • Removed unused peer dependencies from some package to reduce peer dependencies warnings.
  • There are some package in this repo that do not specify itwinjs-core and AppUI 5.0 in their peer deps and produce peer dep warning in test-viewer: @itwin/reports-config-widget-react, @itwin/one-click-lca-react, @itwin/grouping-mapping-widget, @itwin/ec3-widget-react

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tree-Widget benchmark

Benchmark suite Current: 0f2d70e Previous: 96beca0 Deviation Status
categories tree creates initial filtered view for 50k items 2381.88 ms 2452.54 ms -2.88% 〰️
categories tree creates initial filtered view for 50k items (P95 of main thread blocks) 1673 ms 1663 ms 0.60% 〰️
categories tree changing category visibility changes visibility for 50k subCategories 10188.72 ms 9030.65 ms 12.82% 🚨
categories tree changing category visibility changes visibility for 50k subCategories (P95 of main thread blocks) 3913 ms 2554 ms 53.21% 🚨
categories tree changing definition container visibility changes visibility for 50k categories 7305.11 ms 6555.08 ms 11.44% 🚨
categories tree changing definition container visibility changes visibility for 50k categories (P95 of main thread blocks) 4277 ms 3831 ms 11.64% 🚨
models tree creates initial filtered view for 50k target items 2290.93 ms 2262.07 ms 1.28% 〰️
models tree creates initial filtered view for 50k target items (P95 of main thread blocks) 344 ms 289 ms 19.03% 🚨
models tree changing model visibility changes visibility for 50k elements 47296.71 ms 50003.36 ms -5.41% 〰️
models tree changing model visibility changes visibility for 50k elements (P95 of main thread blocks) 266 ms 279 ms -4.66% 〰️
models tree changing category visibility changes visibility for 50k elements 46957.58 ms 46806.21 ms 0.32% 〰️
models tree changing category visibility changes visibility for 50k elements (P95 of main thread blocks) 261 ms 239 ms 9.21% 〰️
models tree changing per-model-category override changes visibility for 50k elements 47139.3 ms 48085.98 ms -1.97% 〰️
models tree changing per-model-category override changes visibility for 50k elements (P95 of main thread blocks) 257 ms 243 ms 5.76% 〰️
models tree changing element visibility changes only parent nodes visibility with 50k elements 48420.67 ms 49137.77 ms -1.46% 〰️
models tree changing element visibility changes only parent nodes visibility with 50k elements (P95 of main thread blocks) 482 ms 476 ms 1.26% 〰️

This comment was automatically generated by workflow using github-action-benchmark.

@saskliutas
saskliutas merged commit 358152a into master Jul 4, 2025
14 checks passed
@saskliutas
saskliutas deleted the test-viewer-bump branch July 4, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants