Skip to content

Consistently propagate intersectionState in relations - #52392

Merged
Anders Hejlsberg (ahejlsberg) merged 7 commits into
mainfrom
fix52252
Feb 2, 2023
Merged

Consistently propagate intersectionState in relations#52392
Anders Hejlsberg (ahejlsberg) merged 7 commits into
mainfrom
fix52252

Conversation

@ahejlsberg

@ahejlsbergAnders Hejlsberg (ahejlsberg) commented Jan 24, 2023

Copy link
Copy Markdown
Member

Fixes#52252.
Fixes#52397.

@ahejlsberg

Copy link
Copy Markdown
MemberAuthor

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 24, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the extended test suite on this PR at 2c938a4. You can monitor the build here.

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 24, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the abridged perf test suite on this PR at 2c938a4. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 24, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the diff-based user code test suite on this PR at 2c938a4. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 24, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the diff-based top-repos suite on this PR at 2c938a4. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 24, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the parallelized Definitely Typed test suite on this PR at 2c938a4. You can monitor the build here.

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg) Here are the results of running the user test suite comparing main and refs/pull/52392/merge:

Everything looks good!

@typescript-bot

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've run the RWC suite on this PR - assuming you're on the TS core team, you can view the resulting diff here.

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg)
The results of the perf run you requested are in!

Here they are:

Comparison Report - main..52392

Metricmain52392DeltaBestWorst
Angular - node (v16.17.1, x64)
Memory used359,056k (± 0.01%)359,014k (± 0.01%)-41k (- 0.01%)358,982k359,039k
Parse Time4.19s (± 0.44%)4.18s (± 0.41%)-0.01s (- 0.20%)4.16s4.21s
Bind Time1.22s (± 0.33%)1.22s (± 0.45%)+0.00s (+ 0.27%)1.22s1.23s
Check Time9.55s (± 0.18%)9.54s (± 0.29%)-0.01s (- 0.16%)9.49s9.56s
Emit Time8.08s (± 0.48%)8.11s (± 0.20%)+0.03s (+ 0.39%)8.09s8.13s
Total Time23.04s (± 0.16%)23.05s (± 0.09%)+0.01s (+ 0.04%)23.01s23.07s
Compiler-Unions - node (v16.17.1, x64)
Memory used194,251k (± 0.03%)194,266k (± 0.03%)+15k (+ 0.01%)194,194k194,348k
Parse Time1.81s (± 0.45%)1.82s (± 0.45%)+0.01s (+ 0.55%)1.81s1.83s
Bind Time0.84s (± 1.06%)0.84s (± 0.99%)+0.01s (+ 0.60%)0.84s0.86s
Check Time10.37s (± 0.36%)10.37s (± 0.50%)+0.00s (+ 0.02%)10.32s10.46s
Emit Time3.06s (± 0.77%)3.07s (± 0.76%)+0.01s (+ 0.22%)3.04s3.10s
Total Time16.08s (± 0.19%)16.10s (± 0.22%)+0.02s (+ 0.12%)16.07s16.16s
Monaco - node (v16.17.1, x64)
Memory used343,217k (± 0.01%)343,252k (± 0.01%)+36k (+ 0.01%)343,227k343,277k
Parse Time3.14s (± 0.82%)3.18s (± 1.87%)+0.05s (+ 1.49%)3.12s3.25s
Bind Time1.11s (± 0.89%)1.11s (± 0.46%)+0.00s (+ 0.45%)1.11s1.12s
Check Time7.89s (± 0.25%)7.92s (± 0.39%)+0.03s (+ 0.38%)7.88s7.96s
Emit Time4.57s (± 0.39%)4.54s (± 0.36%)-0.03s (- 0.62%)4.51s4.55s
Total Time16.70s (± 0.24%)16.75s (± 0.27%)+0.05s (+ 0.33%)16.70s16.81s
TFS - node (v16.17.1, x64)
Memory used299,713k (± 0.01%)299,706k (± 0.00%)-7k (- 0.00%)299,685k299,725k
Parse Time2.47s (± 1.40%)2.49s (± 2.22%)+0.02s (+ 0.88%)2.43s2.56s
Bind Time1.25s (± 0.33%)1.26s (± 0.65%)+0.00s (+ 0.40%)1.25s1.27s
Check Time7.53s (± 0.50%)7.52s (± 0.36%)-0.01s (- 0.09%)7.48s7.56s
Emit Time4.24s (± 0.79%)4.23s (± 0.50%)-0.01s (- 0.24%)4.20s4.26s
Total Time15.49s (± 0.38%)15.50s (± 0.42%)+0.01s (+ 0.05%)15.42s15.56s
material-ui - node (v16.17.1, x64)
Memory used476,133k (± 0.02%)476,116k (± 0.01%)-17k (- 0.00%)476,088k476,190k
Parse Time3.72s (± 0.40%)3.72s (± 0.38%)+0.00s (+ 0.09%)3.70s3.74s
Bind Time1.02s (± 0.00%)1.02s (± 0.00%)0.00s ( 0.00%)1.02s1.02s
Check Time18.05s (± 0.13%)18.07s (± 0.22%)+0.03s (+ 0.17%)18.02s18.13s
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)0.00s ( NaN%)0.00s0.00s
Total Time22.78s (± 0.14%)22.82s (± 0.14%)+0.04s (+ 0.17%)22.77s22.84s
xstate - node (v16.17.1, x64)
Memory used546,280k (± 0.03%)558,623k (± 0.02%)+12,344k (+ 2.26%)558,545k558,798k
Parse Time4.76s (± 0.57%)4.78s (± 0.36%)+0.02s (+ 0.42%)4.76s4.80s
Bind Time1.81s (± 4.15%)1.84s (± 0.66%)+0.03s (+ 1.75%)1.83s1.86s
Check Time2.98s (± 2.58%)3.32s (± 0.36%)+0.35s (+11.58%)3.31s3.34s
Emit Time0.09s (± 4.45%)0.09s (± 0.00%)-0.00s (- 1.82%)0.09s0.09s
Total Time9.65s (± 0.47%)10.04s (± 0.24%)+0.39s (+ 4.01%)10.01s10.08s
System
Machine Namets-ci-ubuntu
Platformlinux 5.4.0-135-generic
Architecturex64
Available Memory16 GB
Available Memory15 GB
CPUs4 × Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz
Hosts
  • node (v16.17.1, x64)
Scenarios
  • Angular - node (v16.17.1, x64)
  • Compiler-Unions - node (v16.17.1, x64)
  • Monaco - node (v16.17.1, x64)
  • TFS - node (v16.17.1, x64)
  • material-ui - node (v16.17.1, x64)
  • xstate - node (v16.17.1, x64)
BenchmarkNameIterations
Current523926
Baselinemain6

Developer Information:

Download Benchmark

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg) Here are the results of running the top-repos suite comparing main and refs/pull/52392/merge:

Everything looks good!

@RyanCavanaugh

Copy link
Copy Markdown
Member

I suspect this might fix #52397?

@ahejlsberg

Copy link
Copy Markdown
MemberAuthor

I suspect this might fix #52397?

It does indeed.

@ahejlsberg

Copy link
Copy Markdown
MemberAuthor

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 25, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the diff-based user code test suite on this PR at 02269e4. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 25, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the parallelized Definitely Typed test suite on this PR at 02269e4. You can monitor the build here.

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 25, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the diff-based top-repos suite on this PR at 02269e4. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 25, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the abridged perf test suite on this PR at 02269e4. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 25, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the extended test suite on this PR at 02269e4. You can monitor the build here.

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg) Here are the results of running the user test suite comparing main and refs/pull/52392/merge:

Everything looks good!

@typescript-bot

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've run the RWC suite on this PR - assuming you're on the TS core team, you can view the resulting diff here.

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg)
The results of the perf run you requested are in!

Here they are:

Comparison Report - main..52392

Metricmain52392DeltaBestWorst
Angular - node (v16.17.1, x64)
Memory used359,206k (± 0.01%)359,184k (± 0.01%)-23k (- 0.01%)359,154k359,235k
Parse Time4.17s (± 0.55%)4.19s (± 0.32%)+0.02s (+ 0.44%)4.17s4.21s
Bind Time1.22s (± 0.52%)1.23s (± 0.42%)+0.01s (+ 0.55%)1.22s1.23s
Check Time9.40s (± 0.35%)9.44s (± 0.51%)+0.04s (+ 0.39%)9.38s9.50s
Emit Time8.06s (± 0.65%)8.09s (± 0.60%)+0.03s (+ 0.37%)8.03s8.17s
Total Time22.85s (± 0.25%)22.94s (± 0.43%)+0.09s (+ 0.38%)22.81s23.10s
Compiler-Unions - node (v16.17.1, x64)
Memory used194,205k (± 0.04%)193,874k (± 0.01%)-330k (- 0.17%)193,835k193,910k
Parse Time1.80s (± 0.29%)1.81s (± 0.54%)+0.01s (+ 0.28%)1.80s1.82s
Bind Time0.85s (± 0.00%)0.84s (± 1.24%)-0.01s (- 0.59%)0.83s0.86s
Check Time10.36s (± 0.57%)10.39s (± 0.60%)+0.03s (+ 0.32%)10.34s10.51s
Emit Time3.06s (± 1.52%)3.07s (± 0.48%)+0.01s (+ 0.27%)3.05s3.09s
Total Time16.07s (± 0.52%)16.12s (± 0.38%)+0.05s (+ 0.29%)16.05s16.23s
Monaco - node (v16.17.1, x64)
Memory used343,261k (± 0.01%)343,271k (± 0.01%)+10k (+ 0.00%)343,230k343,312k
Parse Time3.15s (± 1.40%)3.17s (± 0.88%)+0.02s (+ 0.74%)3.14s3.22s
Bind Time1.11s (± 0.37%)1.11s (± 0.75%)+0.01s (+ 0.60%)1.11s1.13s
Check Time7.71s (± 0.07%)7.75s (± 0.31%)+0.04s (+ 0.52%)7.73s7.80s
Emit Time4.50s (± 0.77%)4.54s (± 0.43%)+0.04s (+ 0.89%)4.52s4.57s
Total Time16.48s (± 0.22%)16.58s (± 0.26%)+0.11s (+ 0.65%)16.55s16.65s
TFS - node (v16.17.1, x64)
Memory used299,768k (± 0.01%)299,743k (± 0.00%)-25k (- 0.01%)299,727k299,760k
Parse Time2.49s (± 1.47%)2.47s (± 0.98%)-0.01s (- 0.60%)2.43s2.49s
Bind Time1.26s (± 0.41%)1.26s (± 0.60%)+0.00s (+ 0.13%)1.25s1.27s
Check Time7.18s (± 0.20%)7.21s (± 0.42%)+0.03s (+ 0.39%)7.17s7.25s
Emit Time4.23s (± 0.83%)4.22s (± 0.53%)-0.00s (- 0.04%)4.20s4.26s
Total Time15.15s (± 0.30%)15.16s (± 0.27%)+0.01s (+ 0.08%)15.09s15.21s
material-ui - node (v16.17.1, x64)
Memory used476,092k (± 0.02%)476,090k (± 0.01%)-3k (- 0.00%)475,998k476,164k
Parse Time3.70s (± 0.33%)3.72s (± 0.31%)+0.02s (+ 0.41%)3.70s3.73s
Bind Time1.02s (± 0.74%)1.03s (± 0.79%)+0.01s (+ 0.82%)1.02s1.04s
Check Time18.06s (± 0.23%)18.18s (± 0.41%)+0.12s (+ 0.65%)18.10s18.27s
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)0.00s ( NaN%)0.00s0.00s
Total Time22.78s (± 0.18%)22.92s (± 0.32%)+0.14s (+ 0.61%)22.85s23.01s
xstate - node (v16.17.1, x64)
Memory used546,227k (± 0.01%)546,843k (± 0.04%)+616k (+ 0.11%)546,633k547,138k
Parse Time4.75s (± 0.25%)4.77s (± 0.37%)+0.03s (+ 0.56%)4.75s4.80s
Bind Time1.81s (± 3.84%)1.76s (± 5.75%)-0.05s (- 2.94%)1.66s1.87s
Check Time3.00s (± 2.69%)3.11s (± 3.48%)+0.11s (+ 3.73%)3.00s3.21s
Emit Time0.09s (± 5.53%)0.09s (± 0.00%)🟩-0.00s (- 3.57%)0.09s0.09s
Total Time9.65s (± 0.19%)9.73s (± 0.26%)+0.09s (+ 0.92%)9.71s9.77s
System
Machine Namets-ci-ubuntu
Platformlinux 5.4.0-135-generic
Architecturex64
Available Memory16 GB
Available Memory15 GB
CPUs4 × Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz
Hosts
  • node (v16.17.1, x64)
Scenarios
  • Angular - node (v16.17.1, x64)
  • Compiler-Unions - node (v16.17.1, x64)
  • Monaco - node (v16.17.1, x64)
  • TFS - node (v16.17.1, x64)
  • material-ui - node (v16.17.1, x64)
  • xstate - node (v16.17.1, x64)
BenchmarkNameIterations
Current523926
Baselinemain6

Developer Information:

Download Benchmark

@ahejlsberg

Copy link
Copy Markdown
MemberAuthor

TypeScript Bot (@typescript-bot) perf test faster

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 28, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the diff-based top-repos suite on this PR at c930975. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 28, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the extended test suite on this PR at c930975. You can monitor the build here.

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 28, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the abridged perf test suite on this PR at c930975. You can monitor the build here.

Update: The results are in!

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg) Here are the results of running the user test suite comparing main and refs/pull/52392/merge:

Everything looks good!

@typescript-bot

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've run the RWC suite on this PR - assuming you're on the TS core team, you can view the resulting diff here.

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg)
The results of the perf run you requested are in!

Here they are:

Comparison Report - main..52392

Metricmain52392DeltaBestWorst
Angular - node (v16.17.1, x64)
Memory used359,204k (± 0.01%)359,171k (± 0.00%)-34k (- 0.01%)359,154k359,189k
Parse Time4.19s (± 0.36%)4.19s (± 0.25%)0.00s ( 0.00%)4.18s4.21s
Bind Time1.22s (± 0.67%)1.23s (± 0.42%)+0.00s (+ 0.27%)1.22s1.23s
Check Time9.41s (± 0.59%)9.44s (± 0.36%)+0.03s (+ 0.30%)9.40s9.49s
Emit Time8.05s (± 0.43%)8.07s (± 0.55%)+0.03s (+ 0.31%)8.00s8.12s
Total Time22.87s (± 0.21%)22.93s (± 0.29%)+0.06s (+ 0.26%)22.87s23.04s
Compiler-Unions - node (v16.17.1, x64)
Memory used194,243k (± 0.04%)194,502k (± 0.72%)+259k (+ 0.13%)193,857k197,371k
Parse Time1.80s (± 0.45%)1.81s (± 0.57%)+0.01s (+ 0.55%)1.80s1.83s
Bind Time0.84s (± 0.61%)0.85s (± 1.16%)+0.01s (+ 0.59%)0.84s0.86s
Check Time10.39s (± 0.83%)10.49s (± 0.67%)+0.10s (+ 0.96%)10.38s10.57s
Emit Time3.09s (± 1.58%)3.06s (± 0.95%)-0.03s (- 0.97%)3.02s3.10s
Total Time16.12s (± 0.79%)16.21s (± 0.56%)+0.09s (+ 0.53%)16.09s16.35s
Monaco - node (v16.17.1, x64)
Memory used343,255k (± 0.01%)343,268k (± 0.01%)+13k (+ 0.00%)343,250k343,316k
Parse Time3.14s (± 0.96%)3.20s (± 1.09%)+0.06s (+ 1.96%)3.16s3.25s
Bind Time1.11s (± 0.94%)1.11s (± 0.46%)-0.00s (- 0.15%)1.11s1.12s
Check Time7.73s (± 0.21%)7.73s (± 0.34%)+0.01s (+ 0.11%)7.69s7.77s
Emit Time4.53s (± 0.46%)4.51s (± 0.68%)-0.01s (- 0.26%)4.48s4.56s
Total Time16.51s (± 0.30%)16.57s (± 0.45%)+0.06s (+ 0.38%)16.48s16.65s
TFS - node (v16.17.1, x64)
Memory used299,757k (± 0.01%)299,750k (± 0.01%)-8k (- 0.00%)299,736k299,772k
Parse Time2.47s (± 1.11%)2.47s (± 1.11%)+0.00s (+ 0.14%)2.45s2.51s
Bind Time1.25s (± 0.82%)1.25s (± 0.67%)+0.00s (+ 0.13%)1.24s1.26s
Check Time7.20s (± 0.50%)7.18s (± 0.23%)-0.02s (- 0.25%)7.16s7.21s
Emit Time4.23s (± 0.96%)4.22s (± 0.46%)-0.01s (- 0.28%)4.20s4.25s
Total Time15.15s (± 0.35%)15.13s (± 0.39%)-0.03s (- 0.18%)15.07s15.24s
material-ui - node (v16.17.1, x64)
Memory used476,077k (± 0.02%)476,092k (± 0.01%)+15k (+ 0.00%)476,052k476,125k
Parse Time3.73s (± 0.44%)3.73s (± 0.28%)-0.00s (- 0.04%)3.71s3.74s
Bind Time1.02s (± 0.53%)1.02s (± 0.80%)-0.00s (- 0.16%)1.01s1.03s
Check Time18.13s (± 0.33%)18.17s (± 0.11%)+0.04s (+ 0.19%)18.14s18.20s
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)0.00s ( NaN%)0.00s0.00s
Total Time22.89s (± 0.30%)22.92s (± 0.12%)+0.03s (+ 0.12%)22.88s22.95s
xstate - node (v16.17.1, x64)
Memory used546,316k (± 0.03%)546,726k (± 0.01%)+411k (+ 0.08%)546,575k546,787k
Parse Time4.76s (± 0.47%)4.78s (± 0.45%)+0.02s (+ 0.42%)4.76s4.81s
Bind Time1.85s (± 0.56%)1.75s (± 5.56%)🟩-0.09s (- 5.14%)1.66s1.86s
Check Time2.96s (± 0.35%)3.16s (± 3.54%)+0.19s (+ 6.58%)3.05s3.29s
Emit Time0.09s (± 5.53%)0.09s (± 0.00%)🟩-0.00s (- 3.57%)0.09s0.09s
Total Time9.66s (± 0.23%)9.78s (± 0.57%)+0.11s (+ 1.16%)9.70s9.85s
System
Machine Namets-ci-ubuntu
Platformlinux 5.4.0-135-generic
Architecturex64
Available Memory16 GB
Available Memory15 GB
CPUs4 × Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz
Hosts
  • node (v16.17.1, x64)
Scenarios
  • Angular - node (v16.17.1, x64)
  • Compiler-Unions - node (v16.17.1, x64)
  • Monaco - node (v16.17.1, x64)
  • TFS - node (v16.17.1, x64)
  • material-ui - node (v16.17.1, x64)
  • xstate - node (v16.17.1, x64)
BenchmarkNameIterations
Current523926
Baselinemain6

Developer Information:

Download Benchmark

@ahejlsberg

Copy link
Copy Markdown
MemberAuthor

TypeScript Bot (@typescript-bot) perf test faster

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 28, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the abridged perf test suite on this PR at c930975. You can monitor the build here.

Update: The results are in!

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg)
The results of the perf run you requested are in!

Here they are:

Comparison Report - main..52392

Metricmain52392DeltaBestWorst
Angular - node (v16.17.1, x64)
Memory used359,204k (± 0.01%)359,204k (± 0.01%)-1k (- 0.00%)359,186k359,251k
Parse Time4.19s (± 0.36%)4.18s (± 0.65%)-0.01s (- 0.20%)4.17s4.24s
Bind Time1.22s (± 0.67%)1.22s (± 0.42%)-0.00s (- 0.00%)1.22s1.23s
Check Time9.41s (± 0.59%)9.45s (± 0.44%)+0.04s (+ 0.46%)9.40s9.50s
Emit Time8.05s (± 0.43%)8.07s (± 0.56%)+0.02s (+ 0.31%)8.04s8.14s
Total Time22.87s (± 0.21%)22.94s (± 0.27%)+0.06s (+ 0.27%)22.85s23.00s
Compiler-Unions - node (v16.17.1, x64)
Memory used194,243k (± 0.04%)194,559k (± 0.68%)+316k (+ 0.16%)193,961k197,245k
Parse Time1.80s (± 0.45%)1.80s (± 0.29%)0.00s ( 0.00%)1.80s1.81s
Bind Time0.84s (± 0.61%)0.84s (± 0.97%)0.00s ( 0.00%)0.83s0.85s
Check Time10.39s (± 0.83%)10.45s (± 0.66%)+0.06s (+ 0.58%)10.38s10.55s
Emit Time3.09s (± 1.58%)3.06s (± 0.93%)-0.03s (- 1.02%)3.03s3.11s
Total Time16.12s (± 0.79%)16.15s (± 0.52%)+0.03s (+ 0.18%)16.04s16.25s
Monaco - node (v16.17.1, x64)
Memory used343,255k (± 0.01%)343,270k (± 0.01%)+15k (+ 0.00%)343,248k343,307k
Parse Time3.14s (± 0.96%)3.17s (± 1.15%)+0.03s (+ 0.90%)3.14s3.24s
Bind Time1.11s (± 0.94%)1.11s (± 0.57%)-0.00s (- 0.45%)1.10s1.12s
Check Time7.73s (± 0.21%)7.75s (± 0.46%)+0.02s (+ 0.32%)7.72s7.81s
Emit Time4.53s (± 0.46%)4.53s (± 0.65%)-0.00s (- 0.04%)4.49s4.56s
Total Time16.51s (± 0.30%)16.56s (± 0.47%)+0.05s (+ 0.31%)16.47s16.67s
TFS - node (v16.17.1, x64)
Memory used299,757k (± 0.01%)299,755k (± 0.01%)-3k (- 0.00%)299,713k299,786k
Parse Time2.47s (± 1.11%)2.47s (± 1.34%)+0.01s (+ 0.27%)2.44s2.53s
Bind Time1.25s (± 0.82%)1.25s (± 1.01%)-0.00s (- 0.27%)1.24s1.27s
Check Time7.20s (± 0.50%)7.20s (± 0.35%)-0.00s (- 0.02%)7.17s7.23s
Emit Time4.23s (± 0.96%)4.21s (± 0.44%)-0.03s (- 0.59%)4.18s4.22s
Total Time15.15s (± 0.35%)15.13s (± 0.34%)-0.02s (- 0.14%)15.07s15.19s
material-ui - node (v16.17.1, x64)
Memory used476,077k (± 0.02%)476,107k (± 0.00%)+30k (+ 0.01%)476,081k476,140k
Parse Time3.73s (± 0.44%)3.72s (± 0.22%)-0.01s (- 0.31%)3.70s3.72s
Bind Time1.02s (± 0.53%)1.02s (± 0.00%)-0.01s (- 0.49%)1.02s1.02s
Check Time18.13s (± 0.33%)18.17s (± 0.38%)+0.04s (+ 0.23%)18.09s18.28s
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)0.00s ( NaN%)0.00s0.00s
Total Time22.89s (± 0.30%)22.91s (± 0.32%)+0.02s (+ 0.11%)22.83s23.03s
xstate - node (v16.17.1, x64)
Memory used546,316k (± 0.03%)546,792k (± 0.02%)+476k (+ 0.09%)546,677k546,912k
Parse Time4.76s (± 0.47%)4.76s (± 0.59%)-0.01s (- 0.10%)4.72s4.79s
Bind Time1.85s (± 0.56%)1.83s (± 0.44%)-0.01s (- 0.72%)1.82s1.84s
Check Time2.96s (± 0.35%)3.04s (± 1.17%)+0.07s (+ 2.47%)3.00s3.09s
Emit Time0.09s (± 5.53%)0.09s (± 0.00%)🟩-0.00s (- 3.57%)0.09s0.09s
Total Time9.66s (± 0.23%)9.72s (± 0.51%)+0.06s (+ 0.59%)9.65s9.79s
System
Machine Namets-ci-ubuntu
Platformlinux 5.4.0-135-generic
Architecturex64
Available Memory16 GB
Available Memory15 GB
CPUs4 × Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz
Hosts
  • node (v16.17.1, x64)
Scenarios
  • Angular - node (v16.17.1, x64)
  • Compiler-Unions - node (v16.17.1, x64)
  • Monaco - node (v16.17.1, x64)
  • TFS - node (v16.17.1, x64)
  • material-ui - node (v16.17.1, x64)
  • xstate - node (v16.17.1, x64)
BenchmarkNameIterations
Current523926
Baselinemain6

Developer Information:

Download Benchmark

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg) Here are the results of running the top-repos suite comparing main and refs/pull/52392/merge:

Everything looks good!

@ahejlsberg

Copy link
Copy Markdown
MemberAuthor

Everything looks good with the latest commit to handle intersections in isDeeplyNestedType.

}

function propertiesRelatedTo(source: Type, target: Type, reportErrors: boolean, excludedProperties: Set<__String> | undefined, intersectionState: IntersectionState): Ternary {
function propertiesRelatedTo(source: Type, target: Type, reportErrors: boolean, excludedProperties: Set<__String> | undefined, optionalsOnly: boolean, intersectionState: IntersectionState): Ternary {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Rather than add a new boolean parameter to this function that we have to pass everywhere, could we not use the existing excludedProperties parameter to exclude all the non-optional properties at the single callsite we wanted to?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Yeah, I considered that, but didn't want the overhead of constructing a new set, and the function is only called in three places.

@microsoftMicrosoft (microsoft) locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: TeamFor Uncommitted BugPR for untriaged, rejected, closed or missing bug

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Deep intersections stopped working in 4.9 Intersection types not merged in 4.9.4

6 participants

@ahejlsberg@typescript-bot@RyanCavanaugh@weswigham@jakebailey@sandersn