Skip to content

Conversation

@ahejlsberg
Copy link
Member

Fixes#42317.

@typescript-bottypescript-bot added Author: Team For Milestone Bug PRs that fix a bug with a specific milestone labels Jan 15, 2021
@ahejlsberg
Copy link
MemberAuthor

@typescript-bot test this
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this faster

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 15, 2021

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

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 15, 2021

Heya @ahejlsberg, I've started to run the parallelized community code test suite on this PR at efd7bc9. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 15, 2021

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

Update: The results are in!

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 15, 2021

Heya @ahejlsberg, I've started to run the parallelized Definitely Typed test suite on this PR at efd7bc9. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

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

Here they are:

Comparison Report - master..42365

Metricmaster42365DeltaBestWorst
Angular - node (v14.15.1, x64)
Memory used322,723k (± 0.01%)322,699k (± 0.01%)-24k (- 0.01%)322,630k322,751k
Parse Time2.00s (± 0.47%)2.01s (± 0.53%)+0.01s (+ 0.45%)1.99s2.03s
Bind Time0.85s (± 0.35%)0.85s (± 0.39%)-0.00s (- 0.23%)0.84s0.86s
Check Time4.89s (± 0.33%)4.90s (± 0.34%)+0.02s (+ 0.35%)4.87s4.95s
Emit Time5.52s (± 0.25%)5.55s (± 0.76%)+0.02s (+ 0.42%)5.48s5.67s
Total Time13.26s (± 0.22%)13.31s (± 0.33%)+0.05s (+ 0.35%)13.23s13.44s
Compiler-Unions - node (v14.15.1, x64)
Memory used201,679k (± 0.53%)200,445k (± 0.37%)-1,234k (- 0.61%)199,610k203,371k
Parse Time0.82s (± 0.60%)0.82s (± 0.46%)-0.00s (- 0.12%)0.81s0.82s
Bind Time0.53s (± 0.84%)0.53s (± 0.65%)-0.01s (- 0.94%)0.52s0.53s
Check Time9.75s (± 0.94%)9.86s (± 1.00%)+0.11s (+ 1.11%)9.65s10.07s
Emit Time2.35s (± 1.56%)2.32s (± 0.89%)-0.03s (- 1.11%)2.29s2.37s
Total Time13.45s (± 0.68%)13.53s (± 0.75%)+0.07s (+ 0.52%)13.30s13.69s
Monaco - node (v14.15.1, x64)
Memory used336,819k (± 0.01%)336,812k (± 0.01%)-7k (- 0.00%)336,730k336,876k
Parse Time1.65s (± 0.64%)1.64s (± 0.67%)-0.01s (- 0.61%)1.62s1.66s
Bind Time0.73s (± 0.91%)0.73s (± 0.79%)+0.00s (+ 0.00%)0.72s0.75s
Check Time4.84s (± 0.46%)4.86s (± 0.37%)+0.02s (+ 0.39%)4.82s4.91s
Emit Time2.94s (± 0.65%)2.93s (± 0.69%)-0.01s (- 0.34%)2.88s2.97s
Total Time10.15s (± 0.34%)10.16s (± 0.33%)+0.00s (+ 0.01%)10.09s10.23s
TFS - node (v14.15.1, x64)
Memory used291,598k (± 0.00%)291,585k (± 0.01%)-13k (- 0.00%)291,531k291,625k
Parse Time1.30s (± 0.70%)1.31s (± 1.37%)+0.01s (+ 0.46%)1.27s1.35s
Bind Time0.69s (± 0.43%)0.69s (± 1.16%)-0.00s (- 0.58%)0.67s0.71s
Check Time4.48s (± 0.53%)4.49s (± 0.65%)+0.01s (+ 0.27%)4.44s4.54s
Emit Time3.06s (± 0.73%)3.05s (± 0.50%)-0.01s (- 0.26%)3.03s3.10s
Total Time9.53s (± 0.37%)9.54s (± 0.48%)+0.01s (+ 0.10%)9.44s9.64s
material-ui - node (v14.15.1, x64)
Memory used472,109k (± 0.05%)472,518k (± 0.04%)+409k (+ 0.09%)472,024k472,738k
Parse Time2.13s (± 0.61%)2.14s (± 0.48%)+0.01s (+ 0.61%)2.12s2.17s
Bind Time0.69s (± 0.90%)0.69s (± 0.68%)+0.00s (+ 0.15%)0.68s0.70s
Check Time12.69s (± 0.95%)12.69s (± 0.81%)-0.01s (- 0.06%)12.50s12.96s
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)0.00s ( NaN%)0.00s0.00s
Total Time15.51s (± 0.72%)15.52s (± 0.63%)+0.01s (+ 0.06%)15.34s15.78s
System
Machine Namets-ci-ubuntu
Platformlinux 4.4.0-198-generic
Architecturex64
Available Memory16 GB
Available Memory10 GB
CPUs4 × Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz
Hosts
  • node (v14.15.1, x64)
Scenarios
  • Angular - node (v14.15.1, x64)
  • Compiler-Unions - node (v14.15.1, x64)
  • Monaco - node (v14.15.1, x64)
  • TFS - node (v14.15.1, x64)
  • material-ui - node (v14.15.1, x64)
BenchmarkNameIterations
Current4236510
Baselinemaster10

@typescript-bot
Copy link
Collaborator

The user suite test run you requested has finished and failed. I've opened a PR with the baseline diff from master.

@ahejlsbergahejlsberg merged commit 70c82ca into masterJan 16, 2021
@ahejlsbergahejlsberg deleted the fix42317 branch January 16, 2021 02:01
@AndaristAndarist mentioned this pull request Mar 30, 2025
@microsoftmicrosoft locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: TeamFor Milestone BugPRs that fix a bug with a specific milestone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

jquery has batch-only inference error after re-aliasing support (##42284)

5 participants

@ahejlsberg@typescript-bot@sandersn@DanielRosenwasser