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

Combined update graph experiments #60800

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

Conversation

DanielRosenwasser
Copy link
Member

Merges #60754 and #60755 to see the combined effect on addressing #60633.

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Dec 17, 2024
@DanielRosenwasser
Copy link
Member Author

@typescript-bot pack this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Dec 17, 2024

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
pack this ✅ Started ✅ Results

@typescript-bot
Copy link
Collaborator

typescript-bot commented Dec 17, 2024

Hey @DanielRosenwasser, I've packed this into an installable tgz. You can install it for testing by referencing it in your package.json like so:

{
    "devDependencies": {
        "typescript": "https://typescript.visualstudio.com/cf7ac146-d525-443c-b23c-0d58337efebc/_apis/build/builds/164413/artifacts?artifactName=tgz&fileId=7F1016401FADCAF8EABD4FD550DAA32A0C47E4C9963BB5155861BA888CA2485802&fileName=/typescript-5.8.0-insiders.20241217.tgz"
    }
}

and then running npm install.


There is also a playground for this build and an npm module you can use via "typescript": "npm:@typescript-deploys/pr-build@5.8.0-pr-60800-2".;

@DanielRosenwasser
Copy link
Member Author

@typescript-bot perf test this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Dec 17, 2024

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
perf test this ✅ Started 👀 Results

@typescript-bot
Copy link
Collaborator

@DanielRosenwasser
The results of the perf run you requested are in!

Here they are:

tsc

Comparison Report - baseline..pr
Metric baseline pr Delta Best Worst p-value
Compiler-Unions - node (v18.15.0, x64)
Errors 34 34 ~ ~ ~ p=1.000 n=6
Symbols 62,363 62,363 ~ ~ ~ p=1.000 n=6
Types 50,395 50,395 ~ ~ ~ p=1.000 n=6
Memory used 195,565k (± 0.95%) 196,716k (± 0.09%) ~ 196,490k 196,854k p=0.471 n=6
Parse Time 1.60s (± 2.42%) 1.59s (± 0.52%) ~ 1.59s 1.61s p=0.513 n=6
Bind Time 0.86s (± 0.94%) 0.86s (± 1.36%) ~ 0.85s 0.88s p=0.315 n=6
Check Time 11.78s (± 0.72%) 11.74s (± 0.27%) ~ 11.70s 11.78s p=0.335 n=6
Emit Time 3.27s (± 1.30%) 3.39s (± 3.95%) ~ 3.27s 3.58s p=0.078 n=6
Total Time 17.51s (± 0.65%) 17.59s (± 0.69%) ~ 17.48s 17.79s p=0.172 n=6
angular-1 - node (v18.15.0, x64)
Errors 37 37 ~ ~ ~ p=1.000 n=6
Symbols 947,936 947,936 ~ ~ ~ p=1.000 n=6
Types 410,955 410,955 ~ ~ ~ p=1.000 n=6
Memory used 1,225,998k (± 0.00%) 1,225,824k (± 0.01%) -174k (- 0.01%) 1,225,707k 1,225,915k p=0.005 n=6
Parse Time 8.02s (± 0.92%) 8.09s (± 0.82%) +0.07s (+ 0.85%) 8.03s 8.21s p=0.045 n=6
Bind Time 2.27s (± 0.71%) 2.29s (± 0.66%) +0.03s (+ 1.18%) 2.28s 2.32s p=0.016 n=6
Check Time 38.21s (± 0.30%) 38.28s (± 0.19%) ~ 38.21s 38.38s p=0.228 n=6
Emit Time 18.27s (± 0.42%) 18.29s (± 0.27%) ~ 18.22s 18.34s p=0.574 n=6
Total Time 66.76s (± 0.30%) 66.96s (± 0.10%) ~ 66.87s 67.02s p=0.065 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,448,617 2,448,617 ~ ~ ~ p=1.000 n=6
Types 896,282 896,282 ~ ~ ~ p=1.000 n=6
Memory used 2,320,747k (± 0.00%) 2,320,028k (± 0.01%) -720k (- 0.03%) 2,319,804k 2,320,194k p=0.005 n=6
Parse Time 11.33s (± 0.58%) 11.36s (± 0.38%) ~ 11.31s 11.43s p=0.470 n=6
Bind Time 2.66s (± 0.51%) 2.65s (± 0.88%) ~ 2.62s 2.69s p=0.869 n=6
Check Time 88.72s (± 2.34%) 89.93s (± 1.46%) ~ 87.97s 91.70s p=0.128 n=6
Emit Time 0.34s (± 2.90%) 0.36s (± 3.26%) 🔻+0.02s (+ 5.91%) 0.35s 0.38s p=0.015 n=6
Total Time 103.04s (± 1.98%) 104.30s (± 1.25%) ~ 102.40s 106.03s p=0.128 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,225,390 1,225,444 +54 (+ 0.00%) ~ ~ p=0.001 n=6
Types 266,584 266,595 +11 (+ 0.00%) ~ ~ p=0.001 n=6
Memory used 2,842,573k (±13.19%) 2,842,940k (±13.21%) ~ 2,357,394k 3,086,316k p=0.810 n=6
Parse Time 6.78s (± 1.66%) 6.69s (± 2.08%) ~ 6.50s 6.80s p=0.261 n=6
Bind Time 2.13s (± 1.09%) 2.15s (± 1.55%) ~ 2.10s 2.19s p=0.470 n=6
Check Time 42.75s (± 0.51%) 42.72s (± 0.69%) ~ 42.28s 43.05s p=1.000 n=6
Emit Time 3.46s (± 3.10%) 3.44s (± 1.45%) ~ 3.39s 3.51s p=0.748 n=6
Total Time 55.13s (± 0.71%) 55.01s (± 0.74%) ~ 54.36s 55.43s p=0.575 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,225,390 1,225,444 +54 (+ 0.00%) ~ ~ p=0.001 n=6
Types 266,584 266,595 +11 (+ 0.00%) ~ ~ p=0.001 n=6
Memory used 3,150,446k (± 0.01%) 3,029,884k (± 9.78%) ~ 2,424,404k 3,151,951k p=0.936 n=6
Parse Time 7.03s (± 0.62%) 6.99s (± 2.01%) ~ 6.75s 7.14s p=0.873 n=6
Bind Time 2.13s (± 0.71%) 2.13s (± 1.55%) ~ 2.10s 2.19s p=0.629 n=6
Check Time 43.01s (± 0.36%) 42.83s (± 0.49%) ~ 42.43s 42.99s p=0.173 n=6
Emit Time 3.45s (± 2.03%) 3.46s (± 2.60%) ~ 3.33s 3.57s p=0.810 n=6
Total Time 55.63s (± 0.33%) 55.40s (± 0.58%) ~ 54.80s 55.72s p=0.230 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 262,278 262,299 +21 (+ 0.01%) ~ ~ p=0.001 n=6
Types 106,628 106,636 +8 (+ 0.01%) ~ ~ p=0.001 n=6
Memory used 439,852k (± 0.01%) 440,116k (± 0.01%) +264k (+ 0.06%) 440,076k 440,159k p=0.005 n=6
Parse Time 2.88s (± 0.36%) 2.89s (± 0.86%) ~ 2.86s 2.91s p=0.461 n=6
Bind Time 1.10s 1.10s ~ ~ ~ p=1.000 n=6
Check Time 15.77s (± 0.12%) 15.73s (± 0.20%) -0.04s (- 0.29%) 15.69s 15.77s p=0.028 n=6
Emit Time 1.31s (± 0.92%) 1.30s (± 1.13%) ~ 1.29s 1.33s p=0.133 n=6
Total Time 21.06s (± 0.16%) 21.02s (± 0.18%) ~ 20.97s 21.07s p=0.126 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 70 70 ~ ~ ~ p=1.000 n=6
Symbols 226,062 226,062 ~ ~ ~ p=1.000 n=6
Types 94,488 94,488 ~ ~ ~ p=1.000 n=6
Memory used 371,577k (± 0.01%) 371,598k (± 0.02%) ~ 371,525k 371,699k p=0.748 n=6
Parse Time 2.90s (± 1.33%) 2.89s (± 1.22%) ~ 2.86s 2.94s p=0.686 n=6
Bind Time 1.58s (± 0.65%) 1.60s (± 1.42%) +0.02s (+ 1.48%) 1.58s 1.64s p=0.040 n=6
Check Time 16.46s (± 0.44%) 16.51s (± 0.24%) ~ 16.44s 16.55s p=0.146 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 20.94s (± 0.49%) 21.00s (± 0.33%) ~ 20.92s 21.09s p=0.199 n=6
vscode - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 3,220,282 3,220,282 ~ ~ ~ p=1.000 n=6
Types 1,107,900 1,107,900 ~ ~ ~ p=1.000 n=6
Memory used 3,286,953k (± 0.01%) 3,286,265k (± 0.01%) -688k (- 0.02%) 3,286,012k 3,286,473k p=0.005 n=6
Parse Time 14.09s (± 0.21%) 14.06s (± 0.29%) ~ 14.00s 14.11s p=0.124 n=6
Bind Time 4.49s (± 0.44%) 4.48s (± 0.39%) ~ 4.46s 4.51s p=0.224 n=6
Check Time 88.78s (± 1.40%) 86.66s (± 0.41%) -2.12s (- 2.39%) 86.26s 87.10s p=0.031 n=6
Emit Time 27.70s (± 1.82%) 28.49s (± 0.41%) +0.79s (+ 2.86%) 28.35s 28.69s p=0.005 n=6
Total Time 135.06s (± 0.67%) 133.69s (± 0.33%) -1.37s (- 1.02%) 133.19s 134.34s p=0.037 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 291,463 291,463 ~ ~ ~ p=1.000 n=6
Types 118,920 118,920 ~ ~ ~ p=1.000 n=6
Memory used 445,314k (± 0.05%) 445,167k (± 0.03%) ~ 444,998k 445,349k p=0.230 n=6
Parse Time 4.11s (± 0.81%) 4.11s (± 1.36%) ~ 4.03s 4.17s p=1.000 n=6
Bind Time 1.77s (± 0.94%) 1.76s (± 1.06%) ~ 1.74s 1.79s p=0.684 n=6
Check Time 18.89s (± 0.31%) 18.77s (± 0.75%) ~ 18.64s 19.04s p=0.065 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 24.77s (± 0.36%) 24.64s (± 0.69%) ~ 24.42s 24.94s p=0.172 n=6
xstate-main - node (v18.15.0, x64)
Errors 5 5 ~ ~ ~ p=1.000 n=6
Symbols 552,233 552,233 ~ ~ ~ p=1.000 n=6
Types 184,971 184,971 ~ ~ ~ p=1.000 n=6
Memory used 492,394k (± 0.04%) 492,235k (± 0.03%) ~ 492,009k 492,369k p=0.066 n=6
Parse Time 3.43s (± 0.65%) 3.43s (± 0.85%) ~ 3.40s 3.47s p=1.000 n=6
Bind Time 1.18s (± 0.54%) 1.19s (± 0.98%) ~ 1.18s 1.21s p=0.070 n=6
Check Time 19.58s (± 0.97%) 19.57s (± 1.04%) ~ 19.39s 19.94s p=0.872 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 24.20s (± 0.77%) 24.19s (± 0.83%) ~ 24.02s 24.56s p=1.000 n=6
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • Compiler-Unions - node (v18.15.0, x64)
  • angular-1 - node (v18.15.0, x64)
  • mui-docs - node (v18.15.0, x64)
  • self-build-src - node (v18.15.0, x64)
  • self-build-src-public-api - node (v18.15.0, x64)
  • self-compiler - node (v18.15.0, x64)
  • ts-pre-modules - node (v18.15.0, x64)
  • vscode - node (v18.15.0, x64)
  • webpack - node (v18.15.0, x64)
  • xstate-main - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

tsserver

Comparison Report - baseline..pr
Metric baseline pr Delta Best Worst p-value
Compiler-UnionsTSServer - node (v18.15.0, x64)
Req 1 - updateOpen 2,821ms (± 1.15%) 2,839ms (± 0.82%) ~ 2,799ms 2,857ms p=0.471 n=6
Req 2 - geterr 7,086ms (±10.93%) 6,613ms (± 9.15%) ~ 6,340ms 7,849ms p=0.128 n=6
Req 3 - references 386ms (± 8.51%) 350ms (± 2.53%) ~ 343ms 367ms p=0.128 n=6
Req 4 - navto 357ms (± 5.27%) 315ms (±13.60%) ~ 273ms 377ms p=0.108 n=6
Req 5 - completionInfo count 1,357 1,357 ~ ~ ~ p=1.000 n=6
Req 5 - completionInfo 125ms (± 9.16%) 126ms (±12.01%) ~ 104ms 140ms p=0.936 n=6
CompilerTSServer - node (v18.15.0, x64)
Req 1 - updateOpen 3,152ms (± 9.20%) 2,988ms (± 0.50%) ~ 2,972ms 3,009ms p=0.470 n=6
Req 2 - geterr 5,582ms (±10.27%) 5,169ms (±11.64%) ~ 4,776ms 5,968ms p=0.093 n=6
Req 3 - references 425ms (± 9.74%) 433ms (± 7.65%) ~ 370ms 465ms p=1.000 n=6
Req 4 - navto 375ms (± 1.75%) 379ms (± 3.50%) ~ 355ms 390ms p=0.230 n=6
Req 5 - completionInfo count 1,519 1,519 ~ ~ ~ p=1.000 n=6
Req 5 - completionInfo 117ms (± 9.47%) 116ms (± 7.86%) ~ 106ms 128ms p=0.936 n=6
xstate-main-1-tsserver - node (v18.15.0, x64)
Req 1 - updateOpen 6,507ms (± 0.41%) 6,803ms (± 6.07%) 🔻+296ms (+ 4.55%) 6,482ms 7,345ms p=0.045 n=6
Req 2 - geterr 1,552ms (±11.52%) 1,613ms (±11.06%) ~ 1,377ms 1,757ms p=0.298 n=6
Req 3 - references 120ms (±12.89%) 110ms (± 7.72%) ~ 93ms 115ms p=0.227 n=6
Req 4 - navto 611ms (± 2.70%) 602ms (± 2.61%) ~ 582ms 630ms p=0.226 n=6
Req 5 - completionInfo count 3,450 3,450 ~ ~ ~ p=1.000 n=6
Req 5 - completionInfo 1,324ms (± 1.66%) 1,344ms (± 2.03%) ~ 1,300ms 1,366ms p=0.199 n=6
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • CompilerTSServer - node (v18.15.0, x64)
  • Compiler-UnionsTSServer - node (v18.15.0, x64)
  • xstate-main-1-tsserver - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

startup

Comparison Report - baseline..pr
Metric baseline pr Delta Best Worst p-value
tsc-startup - node (v18.15.0, x64)
Execution time 184.78ms (± 0.17%) 184.67ms (± 0.19%) -0.12ms (- 0.06%) 182.44ms 188.12ms p=0.000 n=600
tsserver-startup - node (v18.15.0, x64)
Execution time 287.76ms (± 0.30%) 287.80ms (± 0.29%) ~ 280.42ms 291.49ms p=0.301 n=600
tsserverlibrary-startup - node (v18.15.0, x64)
Execution time 272.28ms (± 0.30%) 272.22ms (± 0.32%) ~ 264.95ms 284.87ms p=0.092 n=600
typescript-startup - node (v18.15.0, x64)
Execution time 272.79ms (± 0.34%) 272.62ms (± 0.32%) -0.16ms (- 0.06%) 265.00ms 278.66ms p=0.000 n=600
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • tsc-startup - node (v18.15.0, x64)
  • tsserver-startup - node (v18.15.0, x64)
  • tsserverlibrary-startup - node (v18.15.0, x64)
  • typescript-startup - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

Developer Information:

Download Benchmarks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
Status: Not started
Development

Successfully merging this pull request may close these issues.

3 participants