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

Switch coverage image from ubuntu-22.04 to mariner-2.0 #60541

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

noshlag
Copy link
Member

@noshlag noshlag commented Nov 19, 2024

Fixes #

@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Nov 19, 2024
@typescript-bot
Copy link
Collaborator

This PR doesn't have any linked issues. Please open an issue that references this PR. From there we can discuss and prioritise.

@noshlag
Copy link
Member Author

noshlag commented Nov 19, 2024

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Nov 19, 2024

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

@noshlag noshlag requested a review from jakebailey November 19, 2024 20:09
@typescript-bot
Copy link
Collaborator

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

Everything looks good!

@typescript-bot
Copy link
Collaborator

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

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

@noshlag
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 194,242k (± 0.96%) 194,873k (± 1.03%) ~ 193,021k 196,885k p=0.575 n=6
Parse Time 1.32s (± 0.62%) 1.31s (± 0.92%) ~ 1.29s 1.32s p=0.599 n=6
Bind Time 0.72s 0.72s ~ ~ ~ p=1.000 n=6
Check Time 9.82s (± 0.57%) 9.79s (± 0.52%) ~ 9.73s 9.85s p=0.471 n=6
Emit Time 2.73s (± 0.98%) 2.74s (± 0.58%) ~ 2.71s 2.75s p=0.684 n=6
Total Time 14.58s (± 0.46%) 14.56s (± 0.40%) ~ 14.48s 14.63s p=0.630 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,226,006k (± 0.00%) 1,226,027k (± 0.00%) ~ 1,225,965k 1,226,099k p=0.630 n=6
Parse Time 6.65s (± 0.63%) 6.64s (± 0.99%) ~ 6.54s 6.74s p=0.808 n=6
Bind Time 1.89s (± 0.29%) 1.89s (± 0.40%) ~ 1.88s 1.90s p=0.476 n=6
Check Time 31.96s (± 0.26%) 31.99s (± 0.29%) ~ 31.87s 32.10s p=0.630 n=6
Emit Time 15.19s (± 0.30%) 15.20s (± 0.46%) ~ 15.10s 15.31s p=0.688 n=6
Total Time 55.69s (± 0.17%) 55.71s (± 0.35%) ~ 55.42s 55.87s p=0.470 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,498,273 2,498,273 ~ ~ ~ p=1.000 n=6
Types 908,637 908,637 ~ ~ ~ p=1.000 n=6
Memory used 2,315,878k (± 0.00%) 2,315,893k (± 0.00%) ~ 2,315,852k 2,315,972k p=0.748 n=6
Parse Time 9.31s (± 0.27%) 9.29s (± 0.24%) ~ 9.27s 9.33s p=0.256 n=6
Bind Time 2.17s (± 0.24%) 2.17s ~ ~ ~ p=0.174 n=6
Check Time 74.83s (± 0.42%) 74.90s (± 0.27%) ~ 74.61s 75.18s p=0.575 n=6
Emit Time 0.28s (± 3.68%) 0.29s (± 2.85%) ~ 0.28s 0.30s p=0.865 n=6
Total Time 86.60s (± 0.38%) 86.65s (± 0.24%) ~ 86.33s 86.93s p=0.575 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,225,251 1,225,251 ~ ~ ~ p=1.000 n=6
Types 266,546 266,546 ~ ~ ~ p=1.000 n=6
Memory used 2,354,414k (± 0.04%) 2,414,437k (± 6.14%) ~ 2,353,523k 2,717,306k p=0.689 n=6
Parse Time 5.22s (± 0.65%) 5.24s (± 2.32%) ~ 5.11s 5.45s p=0.689 n=6
Bind Time 1.79s (± 0.89%) 1.77s (± 1.41%) ~ 1.73s 1.80s p=0.419 n=6
Check Time 35.24s (± 0.13%) 35.18s (± 0.74%) ~ 34.69s 35.40s p=0.810 n=6
Emit Time 2.96s (± 2.11%) 2.97s (± 1.18%) ~ 2.93s 3.02s p=0.872 n=6
Total Time 45.22s (± 0.17%) 45.16s (± 0.40%) ~ 44.86s 45.36s p=0.810 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,225,251 1,225,251 ~ ~ ~ p=1.000 n=6
Types 266,546 266,546 ~ ~ ~ p=1.000 n=6
Memory used 3,028,967k (± 9.78%) 3,028,619k (± 9.77%) ~ 2,424,069k 3,150,050k p=0.298 n=6
Parse Time 8.66s (± 1.24%) 8.70s (± 0.99%) ~ 8.56s 8.80s p=0.471 n=6
Bind Time 2.65s (± 1.74%) 2.63s (± 1.24%) ~ 2.59s 2.68s p=0.423 n=6
Check Time 53.19s (± 0.45%) 53.13s (± 0.18%) ~ 53.04s 53.27s p=0.575 n=6
Emit Time 4.31s (± 2.85%) 4.38s (± 2.74%) ~ 4.30s 4.61s p=0.230 n=6
Total Time 68.83s (± 0.43%) 68.83s (± 0.19%) ~ 68.73s 69.07s p=0.810 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 262,223 262,223 ~ ~ ~ p=1.000 n=6
Types 106,607 106,607 ~ ~ ~ p=1.000 n=6
Memory used 439,834k (± 0.02%) 439,769k (± 0.01%) ~ 439,716k 439,801k p=0.378 n=6
Parse Time 3.54s (± 0.99%) 3.55s (± 1.38%) ~ 3.49s 3.62s p=0.809 n=6
Bind Time 1.33s (± 1.06%) 1.33s (± 0.92%) ~ 1.31s 1.34s p=0.564 n=6
Check Time 18.98s (± 0.50%) 18.98s (± 0.48%) ~ 18.85s 19.10s p=0.873 n=6
Emit Time 1.54s (± 1.57%) 1.52s (± 0.50%) ~ 1.51s 1.53s p=0.210 n=6
Total Time 25.38s (± 0.31%) 25.36s (± 0.44%) ~ 25.22s 25.53s p=0.575 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,649k (± 0.04%) 371,633k (± 0.02%) ~ 371,588k 371,759k p=0.936 n=6
Parse Time 2.88s (± 0.74%) 2.90s (± 0.64%) ~ 2.87s 2.92s p=0.289 n=6
Bind Time 1.59s (± 1.32%) 1.59s (± 1.34%) ~ 1.57s 1.63s p=0.934 n=6
Check Time 16.48s (± 0.40%) 16.49s (± 0.28%) ~ 16.41s 16.55s p=0.423 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 20.96s (± 0.34%) 20.98s (± 0.16%) ~ 20.92s 21.01s p=0.198 n=6
vscode - node (v18.15.0, x64)
Errors 4 4 ~ ~ ~ p=1.000 n=6
Symbols 3,154,489 3,154,489 ~ ~ ~ p=1.000 n=6
Types 1,087,206 1,087,206 ~ ~ ~ p=1.000 n=6
Memory used 3,236,915k (± 0.01%) 3,237,049k (± 0.01%) ~ 3,236,402k 3,237,474k p=0.689 n=6
Parse Time 17.32s (± 0.35%) 17.25s (± 0.40%) ~ 17.17s 17.37s p=0.077 n=6
Bind Time 5.42s (± 0.22%) 5.45s (± 0.34%) +0.03s (+ 0.46%) 5.42s 5.47s p=0.033 n=6
Check Time 105.12s (± 1.79%) 104.18s (± 1.50%) ~ 103.00s 107.25s p=0.298 n=6
Emit Time 33.24s (± 5.21%) 32.96s (± 1.40%) ~ 32.09s 33.47s p=0.170 n=6
Total Time 161.12s (± 0.61%) 159.84s (± 0.96%) ~ 158.79s 162.87s p=0.066 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 288,598 288,598 ~ ~ ~ p=1.000 n=6
Types 117,107 117,107 ~ ~ ~ p=1.000 n=6
Memory used 441,024k (± 0.03%) 440,947k (± 0.04%) ~ 440,759k 441,205k p=0.298 n=6
Parse Time 4.10s (± 1.25%) 4.07s (± 0.95%) ~ 4.04s 4.14s p=0.334 n=6
Bind Time 1.75s (± 0.79%) 1.73s (± 1.70%) ~ 1.70s 1.76s p=0.567 n=6
Check Time 18.88s (± 0.32%) 18.88s (± 0.40%) ~ 18.81s 19.01s p=0.872 n=6
Emit Time 0.00s (±244.70%) 0.01s (±109.43%) ~ 0.00s 0.01s p=0.282 n=6
Total Time 24.73s (± 0.44%) 24.68s (± 0.36%) ~ 24.57s 24.82s p=0.470 n=6
xstate-main - node (v18.15.0, x64)
Errors 8 8 ~ ~ ~ p=1.000 n=6
Symbols 551,583 551,583 ~ ~ ~ p=1.000 n=6
Types 184,808 184,808 ~ ~ ~ p=1.000 n=6
Memory used 491,924k (± 0.01%) 491,892k (± 0.01%) ~ 491,817k 491,928k p=0.298 n=6
Parse Time 3.42s (± 0.60%) 3.42s (± 0.50%) ~ 3.39s 3.44s p=1.000 n=6
Bind Time 1.17s (± 1.16%) 1.16s (± 1.14%) ~ 1.14s 1.18s p=0.164 n=6
Check Time 19.61s (± 0.42%) 19.58s (± 0.33%) ~ 19.48s 19.65s p=0.628 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 24.20s (± 0.35%) 24.15s (± 0.25%) ~ 24.06s 24.22s p=0.422 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
Copy link
Collaborator

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

Everything looks good!

@jakebailey jakebailey closed this Dec 4, 2024
@jakebailey jakebailey reopened this Dec 4, 2024
@jakebailey jakebailey closed this Dec 5, 2024
@jakebailey jakebailey reopened this Dec 5, 2024
@jakebailey jakebailey closed this Dec 5, 2024
@jakebailey jakebailey reopened this Dec 5, 2024
Copy link
Member

@jakebailey jakebailey left a comment

Choose a reason for hiding this comment

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

This is all working now.

@jakebailey jakebailey changed the title Switching Image from Ubuntu-22.04 to Mariner-2.0 Switch coverage image from ubuntu-22.04 to mariner-2.0 Dec 5, 2024
@jakebailey jakebailey merged commit edd278a into main Dec 5, 2024
114 of 116 checks passed
@jakebailey jakebailey deleted the naoshl/pipelines/ubuntu-to-mariner branch December 5, 2024 23:16
@jakebailey
Copy link
Member

Merging before full CI just because I know this works and I just dropped ubuntu from the pool (so other PRs would have hung).

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

Successfully merging this pull request may close these issues.

3 participants