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

[lit-html] Make RefOrCallback generic #3969

Merged
merged 2 commits into from Jun 22, 2023
Merged

[lit-html] Make RefOrCallback generic #3969

merged 2 commits into from Jun 22, 2023

Conversation

justinfagnani
Copy link
Collaborator

Make RefOrCallback generic like Ref<T>.

Fixes #3966

@changeset-bot
Copy link

changeset-bot bot commented Jun 21, 2023

🦋 Changeset detected

Latest commit: 52ea0a0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
lit-html Patch
lit Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@justinfagnani justinfagnani changed the title Make RefOrCallback generic [lit-html] Make RefOrCallback generic Jun 21, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 21, 2023

📊 Tachometer Benchmark Results

Summary

nop-update

  • lit-html-kitchen-sink: unsure 🔍 -8% - +7% (-1.79ms - +1.51ms)
    this-change vs tip-of-tree

render

  • lit-element-list: 68.87ms - 72.26ms
  • lit-html-kitchen-sink: unsure 🔍 -7% - +9% (-2.08ms - +2.64ms)
    this-change vs tip-of-tree
  • lit-html-repeat: unsure 🔍 -3% - +4% (-0.26ms - +0.34ms)
    this-change vs tip-of-tree
  • lit-html-template-heavy: unsure 🔍 -3% - +2% (-1.41ms - +1.28ms)
    this-change vs tip-of-tree
  • reactive-element-list: unsure 🔍 -3% - +1% (-1.16ms - +0.51ms)
    this-change vs tip-of-tree

update

  • lit-element-list: 782.15ms - 793.11ms
  • lit-html-kitchen-sink: unsure 🔍 -4% - +7% (-3.15ms - +5.42ms)
    this-change vs tip-of-tree
  • lit-html-repeat: unsure 🔍 -4% - +3% (-11.06ms - +7.18ms)
    this-change vs tip-of-tree
  • lit-html-template-heavy: unsure 🔍 -1% - +2% (-1.50ms - +1.90ms)
    this-change vs tip-of-tree
  • reactive-element-list: unsure 🔍 -0% - +2% (-2.35ms - +12.87ms)
    this-change vs tip-of-tree

update-reflect

  • lit-element-list: 744.55ms - 752.74ms
  • reactive-element-list: unsure 🔍 -1% - +1% (-4.50ms - +8.64ms)
    this-change vs tip-of-tree

Results

lit-element-list

render

VersionAvg timevs
68.87ms - 72.26ms-

update

VersionAvg timevs
782.15ms - 793.11ms-

update-reflect

VersionAvg timevs
744.55ms - 752.74ms-
lit-html-kitchen-sink

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
29.58ms - 32.90ms-unsure 🔍
-7% - +9%
-2.08ms - +2.64ms
unsure 🔍
-6% - +10%
-1.70ms - +2.95ms
tip-of-tree
tip-of-tree
29.29ms - 32.63msunsure 🔍
-8% - +7%
-2.64ms - +2.08ms
-unsure 🔍
-7% - +9%
-1.99ms - +2.68ms
previous-release
previous-release
28.99ms - 32.25msunsure 🔍
-9% - +5%
-2.95ms - +1.70ms
unsure 🔍
-9% - +6%
-2.68ms - +1.99ms
-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
74.76ms - 81.30ms-unsure 🔍
-4% - +7%
-3.15ms - +5.42ms
unsure 🔍
-8% - +3%
-6.49ms - +2.09ms
tip-of-tree
tip-of-tree
74.12ms - 79.67msunsure 🔍
-7% - +4%
-5.42ms - +3.15ms
-unsure 🔍
-9% - +1%
-7.26ms - +0.60ms
previous-release
previous-release
77.44ms - 83.01msunsure 🔍
-3% - +8%
-2.09ms - +6.49ms
unsure 🔍
-1% - +10%
-0.60ms - +7.26ms
-

nop-update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
20.43ms - 23.01ms-unsure 🔍
-8% - +7%
-1.79ms - +1.51ms
unsure 🔍
-15% - +1%
-3.71ms - +0.35ms
tip-of-tree
tip-of-tree
20.83ms - 22.89msunsure 🔍
-7% - +8%
-1.51ms - +1.79ms
-unsure 🔍
-14% - +1%
-3.41ms - +0.33ms
previous-release
previous-release
21.83ms - 24.96msunsure 🔍
-2% - +17%
-0.35ms - +3.71ms
unsure 🔍
-2% - +16%
-0.33ms - +3.41ms
-
lit-html-repeat

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
9.44ms - 9.84ms-unsure 🔍
-3% - +4%
-0.26ms - +0.34ms
unsure 🔍
-3% - +3%
-0.27ms - +0.29ms
tip-of-tree
tip-of-tree
9.38ms - 9.82msunsure 🔍
-4% - +3%
-0.34ms - +0.26ms
-unsure 🔍
-3% - +3%
-0.32ms - +0.26ms
previous-release
previous-release
9.43ms - 9.83msunsure 🔍
-3% - +3%
-0.29ms - +0.27ms
unsure 🔍
-3% - +3%
-0.26ms - +0.32ms
-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
277.11ms - 289.19ms-unsure 🔍
-4% - +3%
-11.06ms - +7.18ms
unsure 🔍
-4% - +2%
-11.48ms - +5.78ms
tip-of-tree
tip-of-tree
278.25ms - 291.93msunsure 🔍
-3% - +4%
-7.18ms - +11.06ms
-unsure 🔍
-4% - +3%
-10.11ms - +8.29ms
previous-release
previous-release
279.84ms - 292.16msunsure 🔍
-2% - +4%
-5.78ms - +11.48ms
unsure 🔍
-3% - +4%
-8.29ms - +10.11ms
-
lit-html-template-heavy

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
50.07ms - 52.06ms-unsure 🔍
-3% - +2%
-1.41ms - +1.28ms
unsure 🔍
-4% - +3%
-1.80ms - +1.37ms
tip-of-tree
tip-of-tree
50.23ms - 52.03msunsure 🔍
-3% - +3%
-1.28ms - +1.41ms
-unsure 🔍
-3% - +3%
-1.68ms - +1.38ms
previous-release
previous-release
50.05ms - 52.51msunsure 🔍
-3% - +4%
-1.37ms - +1.80ms
unsure 🔍
-3% - +3%
-1.38ms - +1.68ms
-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
110.19ms - 112.82ms-unsure 🔍
-1% - +2%
-1.50ms - +1.90ms
unsure 🔍
-2% - +1%
-2.42ms - +1.58ms
tip-of-tree
tip-of-tree
110.23ms - 112.39msunsure 🔍
-2% - +1%
-1.90ms - +1.50ms
-unsure 🔍
-2% - +1%
-2.47ms - +1.23ms
previous-release
previous-release
110.42ms - 113.43msunsure 🔍
-1% - +2%
-1.58ms - +2.42ms
unsure 🔍
-1% - +2%
-1.23ms - +2.47ms
-
reactive-element-list

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
44.18ms - 45.37ms-unsure 🔍
-3% - +1%
-1.16ms - +0.51ms
unsure 🔍
-4% - -0%
-1.71ms - +0.00ms
tip-of-tree
tip-of-tree
44.52ms - 45.68msunsure 🔍
-1% - +3%
-0.51ms - +1.16ms
-unsure 🔍
-3% - +1%
-1.37ms - +0.31ms
previous-release
previous-release
45.02ms - 46.23msunsure 🔍
-0% - +4%
-0.00ms - +1.71ms
unsure 🔍
-1% - +3%
-0.31ms - +1.37ms
-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
810.92ms - 820.70ms-unsure 🔍
-0% - +2%
-2.35ms - +12.87ms
unsure 🔍
-1% - +1%
-6.59ms - +8.80ms
tip-of-tree
tip-of-tree
804.72ms - 816.38msunsure 🔍
-2% - +0%
-12.87ms - +2.35ms
-unsure 🔍
-2% - +1%
-12.48ms - +4.17ms
previous-release
previous-release
808.76ms - 820.64msunsure 🔍
-1% - +1%
-8.80ms - +6.59ms
unsure 🔍
-1% - +2%
-4.17ms - +12.48ms
-

update-reflect

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
788.91ms - 798.05ms-unsure 🔍
-1% - +1%
-4.50ms - +8.64ms
unsure 🔍
-1% - +1%
-6.78ms - +6.58ms
tip-of-tree
tip-of-tree
786.69ms - 796.13msunsure 🔍
-1% - +1%
-8.64ms - +4.50ms
-unsure 🔍
-1% - +1%
-8.96ms - +4.61ms
previous-release
previous-release
788.71ms - 798.45msunsure 🔍
-1% - +1%
-6.58ms - +6.78ms
unsure 🔍
-1% - +1%
-4.61ms - +8.96ms
-

tachometer-reporter-action v2 for Benchmarks

.changeset/smooth-swans-sin.md Outdated Show resolved Hide resolved
Co-authored-by: Augustine Kim <augustinekim@google.com>
@justinfagnani justinfagnani merged commit 7d8d4a1 into main Jun 22, 2023
7 checks passed
@justinfagnani justinfagnani deleted the ref-generics branch June 22, 2023 01:13
@lit-robot lit-robot mentioned this pull request Jul 5, 2023
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.

[lit-html] RefOrCallback should be generic
3 participants