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

[WIP] chore: experiments react compiler #61

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[WIP] chore: experiments react compiler #61

wants to merge 2 commits into from

Conversation

mdjastrzebski
Copy link
Member

Summary

Test plan

@github-actions GitHub Actions
Copy link

Performance Comparison Report

  • Current: 04b1c7a - 2024-06-21 13:28:19Z
  • Baseline: main (9a714e5) - 2024-06-21 13:27:56Z

Significant Changes To Duration

There are no entries

Meaningless Changes To Duration

Show entries
Name Type Duration Count
Web Next.js - AsyncComponent 10 runs render 48.3 ms → 48.2 ms (-0.1 ms, ±0.0%) 5 → 5
Web Next.js - AsyncComponent 50 runs render 48.3 ms → 48.0 ms (-0.3 ms, -0.5%) 5 → 5
Web Next.js - SlowList 100 items render 14.0 ms → 13.9 ms (-0.1 ms, -0.6%) 2 → 2
Show details
Name Type Duration Count
Web Next.js - AsyncComponent 10 runs render Baseline
Mean: 48.3 ms
Stdev: 0.2 ms (0.3%)
Runs: 48.2 48.3 48.4 48.6 48.2 48.3 48.5 48.1 48.3 48.2

Current
Mean: 48.2 ms
Stdev: 0.4 ms (0.9%)
Runs: 48.4 47.6 47.7 47.9 47.9 48.1 48.1 48.8 48.8 48.5		 Baseline
Mean: 5
Stdev: 0 (0.0%)
Runs: 5 5 5 5 5 5 5 5 5 5

Current
Mean: 5
Stdev: 0 (0.0%)
Runs: 5 5 5 5 5 5 5 5 5 5
Web Next.js - AsyncComponent 50 runs render Baseline
Mean: 48.3 ms
Stdev: 0.9 ms (1.9%)
Runs: 48.3 48.0 48.0 48.1 48.1 48.1 48.4 47.8 47.8 48.2 48.1 48.1 52.9 47.9 48.0 48.0 48.0 47.9 48.3 48.2 50.3 48.3 51.1 48.0 47.7 48.1 47.8 48.1 47.9 47.9 48.1 48.0 48.5 47.8 47.7 47.8 48.2 47.7 47.9 47.8 47.9 48.2 48.3 48.2 47.9 47.9 48.1 48.5 49.8 48.3

Current
Mean: 48.0 ms
Stdev: 1.0 ms (2.1%)
Runs: 48.4 48.0 47.9 47.9 48.1 47.8 49.1 48.1 48.5 48.0 48.0 47.6 47.6 48.0 47.6 47.8 47.8 47.8 47.6 47.3 51.8 48.5 47.7 48.0 47.2 47.7 47.5 47.5 47.6 47.0 47.3 47.6 47.4 47.5 47.8 47.5 47.5 48.2 50.1 52.0 48.0 47.9 47.1 47.1 47.3 47.8 47.9 47.7 48.1 50.0		 Baseline
Mean: 5
Stdev: 0 (0.0%)
Runs: 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5

Current
Mean: 5
Stdev: 0 (0.0%)
Runs: 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5
Web Next.js - SlowList 100 items render Baseline
Mean: 14.0 ms
Stdev: 0.1 ms (0.5%)
Runs: 13.9 14.0 14.1 13.9 13.9 14.0 14.0 13.9 13.9 13.8

Current
Mean: 13.9 ms
Stdev: 0.1 ms (1.0%)
Runs: 14.1 14.1 13.7 13.8 13.8 13.9 13.8 13.8 13.8 13.8		 Baseline
Mean: 2
Stdev: 0 (0.0%)
Runs: 2 2 2 2 2 2 2 2 2 2

Current
Mean: 2
Stdev: 0 (0.0%)
Runs: 2 2 2 2 2 2 2 2 2 2

Render Count Changes

There are no entries

Render Issues

Name Initial Updates Redundant Updates
Web Next.js - AsyncComponent 10 runs 1 🔴 -
Web Next.js - AsyncComponent 50 runs 1 🔴 -
Web Next.js - SlowList 100 items 1 🔴 -

Added Scenarios

There are no entries

Removed Scenarios

There are no entries

Generated by 🚫 dangerJS against 6a40979

@github-actions GitHub Actions
Copy link

Performance Comparison Report

  • Current: 04b1c7a - 2024-06-21 13:28:21Z
  • Baseline: main (9a714e5) - 2024-06-21 13:27:58Z

Significant Changes To Duration

There are no entries

Meaningless Changes To Duration

Show entries
Name Type Duration Count
Web Vite - AsyncComponent 10 runs render 50.9 ms → 51.2 ms (+0.3 ms, +0.6%) 5 → 5
Web Vite - AsyncComponent 50 runs render 50.8 ms → 51.1 ms (+0.4 ms, +0.7%) 5 → 5
Web Vite - SlowList 100 items render 14.9 ms → 14.8 ms (-0.1 ms, -0.8%) 2 → 2
Show details
Name Type Duration Count
Web Vite - AsyncComponent 10 runs render Baseline
Mean: 50.9 ms
Stdev: 0.8 ms (1.6%)
Runs: 52.1 50.8 51.0 52.0 51.4 50.1 49.8 50.3 51.1 50.0

Current
Mean: 51.2 ms
Stdev: 0.7 ms (1.4%)
Runs: 50.4 50.6 51.0 51.7 51.3 50.9 50.1 52.2 52.1 51.6		 Baseline
Mean: 5
Stdev: 0 (0.0%)
Runs: 5 5 5 5 5 5 5 5 5 5

Current
Mean: 5
Stdev: 0 (0.0%)
Runs: 5 5 5 5 5 5 5 5 5 5
Web Vite - AsyncComponent 50 runs render Baseline
Mean: 50.8 ms
Stdev: 1.9 ms (3.7%)
Runs: 49.6 49.8 51.3 50.7 50.5 50.0 50.6 50.6 50.7 50.5 50.1 49.6 50.9 51.3 50.5 54.3 50.1 51.5 50.9 51.4 50.3 50.3 49.5 49.8 51.0 50.8 50.7 50.4 50.2 50.7 49.9 50.2 50.2 50.8 50.5 49.6 50.1 50.6 50.2 50.4 49.9 56.7 60.9 49.5 49.5 49.8 50.8 50.4 49.9 49.7

Current
Mean: 51.1 ms
Stdev: 1.4 ms (2.8%)
Runs: 51.6 51.6 51.6 51.2 50.7 51.2 51.1 51.6 51.3 51.3 50.8 51.1 50.6 52.4 51.2 50.7 50.7 52.3 58.5 55.4 51.7 52.1 51.4 51.2 50.6 49.8 51.8 50.4 50.1 51.6 51.6 50.8 50.8 50.5 51.2 51.0 50.6 50.6 50.9 49.2 49.7 50.0 50.1 50.1 49.6 50.3 50.3 50.2 50.7 50.4		 Baseline
Mean: 5
Stdev: 0 (0.0%)
Runs: 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5

Current
Mean: 5
Stdev: 0 (0.0%)
Runs: 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5
Web Vite - SlowList 100 items render Baseline
Mean: 14.9 ms
Stdev: 0.4 ms (2.4%)
Runs: 15.1 15.2 14.8 14.9 15.0 14.4 14.4 15.4 15.2 14.4

Current
Mean: 14.8 ms
Stdev: 0.2 ms (1.1%)
Runs: 14.8 14.7 14.6 14.8 14.7 14.5 14.9 15.1 14.8 14.8		 Baseline
Mean: 2
Stdev: 0 (0.0%)
Runs: 2 2 2 2 2 2 2 2 2 2

Current
Mean: 2
Stdev: 0 (0.0%)
Runs: 2 2 2 2 2 2 2 2 2 2

Render Count Changes

There are no entries

Render Issues

Name Initial Updates Redundant Updates
Web Vite - AsyncComponent 10 runs 1 🔴 -
Web Vite - AsyncComponent 50 runs 1 🔴 -
Web Vite - SlowList 100 items 1 🔴 -

Added Scenarios

There are no entries

Removed Scenarios

There are no entries

Generated by 🚫 dangerJS against 6a40979

@github-actions GitHub Actions
Copy link

Performance Comparison Report

  • Current: 04b1c7a - 2024-06-21 13:28:41Z
  • Baseline: main (9a714e5) - 2024-06-21 13:28:02Z

Significant Changes To Duration

There are no entries

Meaningless Changes To Duration

Show entries
Name Type Duration Count
RN Expo - AsyncComponent 10 runs render 68.4 ms → 65.3 ms (-3.1 ms, -4.5%) 4 → 4
RN Expo - AsyncComponent 50 runs render 65.4 ms → 65.6 ms (+0.2 ms, ±0.0%) 4 → 4
RN Expo - SlowList 100 items render 17.7 ms → 18.4 ms (+0.7 ms, +4.0%) 2 → 2
Show details
Name Type Duration Count
RN Expo - AsyncComponent 10 runs render Baseline
Mean: 68.4 ms
Stdev: 6.4 ms (9.3%)
Runs: 67 69 68 64 85 71 62 65 67 66

Current
Mean: 65.3 ms
Stdev: 2.8 ms (4.3%)
Runs: 63 67 62 71 63 65 65 63 66 68		 Baseline
Mean: 4
Stdev: 0 (0.0%)
Runs: 4 4 4 4 4 4 4 4 4 4

Current
Mean: 4
Stdev: 0 (0.0%)
Runs: 4 4 4 4 4 4 4 4 4 4
RN Expo - AsyncComponent 50 runs render Baseline
Mean: 65.4 ms
Stdev: 3.8 ms (5.8%)
Runs: 66 65 72 67 69 68 72 66 69 67 59 71 58 71 66 65 68 64 60 62 72 64 65 63 61 71 63 65 67 62 70 61 67 67 67 61 63 66 69 57 64 64 68 63 65 58 69 65 66 64

Current
Mean: 65.6 ms
Stdev: 4.8 ms (7.3%)
Runs: 67 61 69 81 61 68 65 70 69 61 82 63 59 62 68 63 67 66 62 68 76 60 67 70 65 66 65 69 60 61 62 64 60 63 61 68 66 68 65 65 67 63 69 62 64 71 61 64 62 65		 Baseline
Mean: 4
Stdev: 0 (0.0%)
Runs: 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4

Current
Mean: 4
Stdev: 0 (0.0%)
Runs: 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4
RN Expo - SlowList 100 items render Baseline
Mean: 17.7 ms
Stdev: 0.9 ms (5.4%)
Runs: 16 19 18 18 17 18 17 19 17 18

Current
Mean: 18.4 ms
Stdev: 1.0 ms (5.3%)
Runs: 18 20 20 18 18 18 19 18 18 17		 Baseline
Mean: 2
Stdev: 0 (0.0%)
Runs: 2 2 2 2 2 2 2 2 2 2

Current
Mean: 2
Stdev: 0 (0.0%)
Runs: 2 2 2 2 2 2 2 2 2 2

Render Count Changes

There are no entries

Render Issues

Name Initial Updates Redundant Updates
RN Expo - AsyncComponent 10 runs 1 🔴 -
RN Expo - AsyncComponent 50 runs 1 🔴 -
RN Expo - SlowList 100 items 1 🔴 -

Added Scenarios

There are no entries

Removed Scenarios

There are no entries

Generated by 🚫 dangerJS against 6a40979

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@piotrski piotrski Awaiting requested review from piotrski

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mdjastrzebski