TypeScript icon indicating copy to clipboard operation
TypeScript copied to clipboard

Reuse single errored signature candidate as resolved signature

Open Andarist opened this issue 6 months ago • 7 comments

fixes https://github.com/microsoft/TypeScript/issues/61500

Andarist avatar May 30 '25 19:05 Andarist

would it make sense to emit a warning when any is inferred in these partially annotated functions, to help catch unintended inference?

Mateusnasciment avatar Jun 03 '25 16:06 Mateusnasciment

@typescript-bot test it

jakebailey avatar Jun 06 '25 19:06 jakebailey

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

Command Status Results
test top400 ✅ Started ✅ Results
user test this ✅ Started ✅ Results
run dt ✅ Started ✅ Results
perf test this faster ✅ Started 👀 Results

typescript-bot avatar Jun 06 '25 19:06 typescript-bot

Hey @jakebailey, the results of running the DT tests are ready.

Everything looks the same!

You can check the log here.

typescript-bot avatar Jun 06 '25 19:06 typescript-bot

@jakebailey Here are the results of running the user tests with tsc comparing main and refs/pull/61787/merge:

Everything looks good!

typescript-bot avatar Jun 06 '25 19:06 typescript-bot

@jakebailey 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,390 62,390 ~ ~ ~ p=1.000 n=6
Types 50,395 50,395 ~ ~ ~ p=1.000 n=6
Memory used 196,572k (± 0.08%) 196,086k (± 0.78%) ~ 192,977k 196,729k p=0.298 n=6
Parse Time 1.59s (± 1.23%) 1.59s (± 1.24%) ~ 1.55s 1.60s p=0.357 n=6
Bind Time 0.88s (± 0.96%) 0.88s (± 1.61%) ~ 0.86s 0.90s p=0.548 n=6
Check Time 11.83s (± 0.69%) 11.81s (± 0.84%) ~ 11.70s 11.97s p=0.748 n=6
Emit Time 3.39s (± 3.47%) 3.37s (± 2.59%) ~ 3.29s 3.54s p=1.000 n=6
Total Time 17.69s (± 0.51%) 17.64s (± 0.74%) ~ 17.45s 17.82s p=0.748 n=6
angular-1 - node (v18.15.0, x64)
Errors 56 56 ~ ~ ~ p=1.000 n=6
Symbols 949,240 949,228 -12 (- 0.00%) ~ ~ p=0.001 n=6
Types 411,065 411,065 ~ ~ ~ p=1.000 n=6
Memory used 1,225,316k (± 0.01%) 1,225,320k (± 0.00%) ~ 1,225,242k 1,225,396k p=0.810 n=6
Parse Time 8.03s (± 0.75%) 8.01s (± 0.62%) ~ 7.96s 8.08s p=0.748 n=6
Bind Time 2.29s (± 0.51%) 2.30s (± 0.98%) ~ 2.28s 2.33s p=0.730 n=6
Check Time 38.10s (± 0.39%) 38.19s (± 0.29%) ~ 38.00s 38.29s p=0.298 n=6
Emit Time 17.90s (± 0.66%) 17.95s (± 0.69%) ~ 17.77s 18.14s p=0.521 n=6
Total Time 66.32s (± 0.32%) 66.45s (± 0.13%) ~ 66.33s 66.53s p=0.229 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,577,125 2,576,904 -221 (- 0.01%) ~ ~ p=0.001 n=6
Types 892,437 892,364 -73 (- 0.01%) ~ ~ p=0.001 n=6
Memory used 2,857,483k (± 0.00%) 2,857,350k (± 0.00%) ~ 2,857,116k 2,857,497k p=0.066 n=6
Parse Time 11.00s (± 0.49%) 10.99s (± 0.25%) ~ 10.97s 11.04s p=0.422 n=6
Bind Time 2.82s (± 0.32%) 2.80s (± 0.93%) ~ 2.78s 2.85s p=0.063 n=6
Check Time 106.36s (± 1.58%) 106.36s (± 1.84%) ~ 104.53s 109.87s p=0.936 n=6
Emit Time 0.38s (± 2.23%) 0.38s (± 2.23%) ~ 0.37s 0.39s p=0.796 n=6
Total Time 120.56s (± 1.37%) 120.53s (± 1.61%) ~ 118.69s 124.02s p=0.936 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,228,872 1,228,874 +2 (+ 0.00%) ~ ~ p=0.001 n=6
Types 267,514 267,514 ~ ~ ~ p=1.000 n=6
Memory used 2,731,011k (±14.66%) 2,852,990k (±13.27%) ~ 2,363,451k 3,099,145k p=0.810 n=6
Parse Time 6.61s (± 1.38%) 6.63s (± 1.27%) ~ 6.51s 6.71s p=0.872 n=6
Bind Time 2.16s (± 0.94%) 2.16s (± 1.30%) ~ 2.11s 2.19s p=0.935 n=6
Check Time 42.88s (± 0.56%) 42.97s (± 0.38%) ~ 42.70s 43.13s p=0.689 n=6
Emit Time 3.52s (± 3.73%) 3.55s (± 4.14%) ~ 3.39s 3.78s p=0.809 n=6
Total Time 55.17s (± 0.71%) 55.31s (± 0.36%) ~ 54.93s 55.46s p=0.689 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,228,872 1,228,874 +2 (+ 0.00%) ~ ~ p=0.001 n=6
Types 267,514 267,514 ~ ~ ~ p=1.000 n=6
Memory used 2,919,449k (±12.86%) 2,919,117k (±12.89%) ~ 2,432,409k 3,163,015k p=0.936 n=6
Parse Time 6.84s (± 1.81%) 6.83s (± 1.28%) ~ 6.70s 6.94s p=0.810 n=6
Bind Time 2.17s (± 1.11%) 2.20s (± 2.42%) ~ 2.14s 2.28s p=0.520 n=6
Check Time 43.01s (± 0.69%) 42.99s (± 0.57%) ~ 42.71s 43.34s p=0.810 n=6
Emit Time 3.49s (± 3.07%) 3.50s (± 2.96%) ~ 3.41s 3.68s p=0.936 n=6
Total Time 55.50s (± 0.65%) 55.52s (± 0.47%) ~ 55.11s 55.84s p=0.936 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 263,510 263,512 +2 (+ 0.00%) ~ ~ p=0.001 n=6
Types 107,227 107,227 ~ ~ ~ p=1.000 n=6
Memory used 442,143k (± 0.01%) 442,144k (± 0.03%) ~ 442,006k 442,356k p=0.575 n=6
Parse Time 3.57s (± 1.19%) 3.52s (± 0.83%) ~ 3.49s 3.56s p=0.091 n=6
Bind Time 1.32s (± 1.55%) 1.32s (± 0.78%) ~ 1.31s 1.34s p=0.565 n=6
Check Time 18.94s (± 0.39%) 18.95s (± 0.36%) ~ 18.82s 19.00s p=0.470 n=6
Emit Time 1.52s (± 0.93%) 1.53s (± 0.64%) ~ 1.51s 1.54s p=0.278 n=6
Total Time 25.34s (± 0.34%) 25.32s (± 0.35%) ~ 25.17s 25.41s p=0.810 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 71 71 ~ ~ ~ p=1.000 n=6
Symbols 225,981 225,981 ~ ~ ~ p=1.000 n=6
Types 94,356 94,356 ~ ~ ~ p=1.000 n=6
Memory used 371,358k (± 0.04%) 371,344k (± 0.04%) ~ 371,188k 371,601k p=0.936 n=6
Parse Time 3.60s (± 0.58%) 3.57s (± 1.13%) ~ 3.54s 3.64s p=0.228 n=6
Bind Time 1.97s (± 0.76%) 1.96s (± 0.72%) ~ 1.94s 1.98s p=0.560 n=6
Check Time 20.48s (± 0.41%) 20.46s (± 0.36%) ~ 20.41s 20.59s p=0.628 n=6
Emit Time 0.00s 0.00s (±244.70%) ~ 0.00s 0.01s p=0.405 n=6
Total Time 26.05s (± 0.26%) 26.00s (± 0.28%) ~ 25.91s 26.12s p=0.149 n=6
vscode - node (v18.15.0, x64)
Errors 38 38 ~ ~ ~ p=1.000 n=6
Symbols 3,495,364 3,495,358 -6 (- 0.00%) ~ ~ p=0.001 n=6
Types 1,180,160 1,180,160 ~ ~ ~ p=1.000 n=6
Memory used 3,536,942k (± 0.00%) 3,536,976k (± 0.00%) ~ 3,536,780k 3,537,098k p=0.575 n=6
Parse Time 12.25s (± 0.41%) 12.25s (± 0.36%) ~ 12.21s 12.33s p=1.000 n=6
Bind Time 3.99s (± 0.40%) 3.99s (± 0.22%) ~ 3.98s 4.00s p=0.933 n=6
Check Time 80.28s (± 0.53%) 80.26s (± 0.34%) ~ 79.95s 80.60s p=0.936 n=6
Emit Time 24.96s (± 0.94%) 24.82s (± 0.44%) ~ 24.65s 24.98s p=0.199 n=6
Total Time 121.47s (± 0.55%) 121.32s (± 0.27%) ~ 120.79s 121.62s p=0.936 n=6
webpack - node (v18.15.0, x64)
Errors 2 2 ~ ~ ~ p=1.000 n=6
Symbols 320,086 320,086 ~ ~ ~ p=1.000 n=6
Types 140,412 140,412 ~ ~ ~ p=1.000 n=6
Memory used 474,279k (± 0.01%) 474,312k (± 0.02%) ~ 474,156k 474,395k p=0.298 n=6
Parse Time 4.31s (± 0.92%) 4.34s (± 0.58%) ~ 4.31s 4.38s p=0.086 n=6
Bind Time 1.76s (± 1.37%) 1.76s (± 1.29%) ~ 1.73s 1.79s p=0.519 n=6
Check Time 20.72s (± 0.53%) 20.74s (± 0.51%) ~ 20.57s 20.83s p=0.810 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 26.80s (± 0.40%) 26.83s (± 0.44%) ~ 26.64s 26.98s p=0.630 n=6
xstate-main - node (v18.15.0, x64)
Errors 5 5 ~ ~ ~ p=1.000 n=6
Symbols 571,458 570,465 -993 (- 0.17%) ~ ~ p=0.001 n=6
Types 191,798 191,234 -564 (- 0.29%) ~ ~ p=0.001 n=6
Memory used 503,354k (± 0.04%) 502,869k (± 0.01%) -485k (- 0.10%) 502,797k 502,991k p=0.005 n=6
Parse Time 3.45s (± 0.83%) 3.44s (± 0.65%) ~ 3.41s 3.47s p=0.418 n=6
Bind Time 1.24s (± 1.33%) 1.24s (± 1.18%) ~ 1.22s 1.26s p=0.514 n=6
Check Time 19.98s (± 3.07%) 19.59s (± 0.23%) ~ 19.54s 19.65s p=0.128 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 24.67s (± 2.41%) 24.27s (± 0.23%) ~ 24.19s 24.36s p=0.092 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
Developer Information:

Download Benchmarks

typescript-bot avatar Jun 06 '25 20:06 typescript-bot

@jakebailey Here are the results of running the top 400 repos with tsc comparing main and refs/pull/61787/merge:

Everything looks good!

typescript-bot avatar Jun 06 '25 21:06 typescript-bot