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

DOM update 2024-04-16 #58211

Merged
merged 2 commits into from
Apr 19, 2024
Merged

DOM update 2024-04-16 #58211

merged 2 commits into from
Apr 19, 2024

Conversation

sandersn
Copy link
Member

@sandersn sandersn commented Apr 16, 2024

For 5.5 beta

  • dom.asyncIterable adds [Symbol.asyncIterator] and values
  • Some additions in Visibility types
  • Minor changes to gamepad types.
  • New ReadableStream iterator options
  • Some properties flip between optional/required
  • Add AbortSignal.any method
  • Add new CSS*Rules
  • Add Document.parseHTMLUnsafe, ShadowRoot.setHTMLUnsafe
  • Minor changes to documentation.
  • HTMLSelectElement.type is now "select-one" | "select-multiple" instead of string
  • new performance type LargestContentfulPaint

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

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Apr 16, 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
perf test this faster ✅ Started 👀 Results

@typescript-bot
Copy link
Collaborator

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

Everything looks good!

@typescript-bot
Copy link
Collaborator

@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
Angular - node (v18.15.0, x64)
Memory used 296,995k (± 0.01%) 297,053k (± 0.00%) +58k (+ 0.02%) 297,041k 297,069k p=0.006 n=6
Parse Time 2.70s (± 0.61%) 2.69s (± 0.72%) ~ 2.67s 2.72s p=0.871 n=6
Bind Time 0.82s (± 0.77%) 0.82s (± 0.63%) ~ 0.81s 0.82s p=0.386 n=6
Check Time 8.33s (± 0.37%) 8.33s (± 0.21%) ~ 8.30s 8.35s p=0.747 n=6
Emit Time 7.06s (± 0.23%) 7.06s (± 0.38%) ~ 7.02s 7.08s p=0.806 n=6
Total Time 18.90s (± 0.23%) 18.90s (± 0.17%) ~ 18.86s 18.94s p=0.808 n=6
Compiler-Unions - node (v18.15.0, x64)
Memory used 193,053k (± 0.94%) 192,438k (± 0.74%) ~ 191,757k 195,314k p=0.471 n=6
Parse Time 1.36s (± 1.21%) 1.36s (± 0.38%) ~ 1.35s 1.36s p=0.794 n=6
Bind Time 0.72s (± 0.00%) 0.72s (± 0.00%) ~ 0.72s 0.72s p=1.000 n=6
Check Time 9.58s (± 0.37%) 9.57s (± 0.64%) ~ 9.47s 9.64s p=0.687 n=6
Emit Time 2.61s (± 0.31%) 2.62s (± 0.48%) ~ 2.60s 2.64s p=0.282 n=6
Total Time 14.26s (± 0.21%) 14.27s (± 0.45%) ~ 14.16s 14.34s p=0.809 n=6
Monaco - node (v18.15.0, x64)
Memory used 348,869k (± 0.00%) 348,851k (± 0.01%) ~ 348,819k 348,868k p=0.127 n=6
Parse Time 2.55s (± 0.54%) 2.56s (± 0.69%) ~ 2.53s 2.58s p=1.000 n=6
Bind Time 0.89s (± 0.46%) 0.89s (± 0.71%) ~ 0.88s 0.90s p=0.673 n=6
Check Time 7.09s (± 0.66%) 7.05s (± 0.59%) ~ 6.99s 7.10s p=0.170 n=6
Emit Time 4.06s (± 0.36%) 4.05s (± 0.43%) ~ 4.03s 4.08s p=0.417 n=6
Total Time 14.60s (± 0.37%) 14.55s (± 0.34%) ~ 14.48s 14.62s p=0.147 n=6
TFS - node (v18.15.0, x64)
Memory used 302,972k (± 0.01%) 302,989k (± 0.01%) ~ 302,969k 303,004k p=0.173 n=6
Parse Time 2.95s (± 1.53%) 2.98s (± 1.32%) ~ 2.92s 3.03s p=0.170 n=6
Bind Time 1.44s (± 0.62%) 1.44s (± 0.44%) ~ 1.43s 1.45s p=1.000 n=6
Check Time 9.38s (± 0.50%) 9.38s (± 0.16%) ~ 9.36s 9.40s p=0.807 n=6
Emit Time 5.32s (± 0.51%) 5.30s (± 0.57%) ~ 5.26s 5.34s p=0.334 n=6
Total Time 19.08s (± 0.49%) 19.10s (± 0.26%) ~ 19.03s 19.17s p=0.629 n=6
material-ui - node (v18.15.0, x64)
Memory used 511,613k (± 0.00%) 511,692k (± 0.01%) +79k (+ 0.02%) 511,650k 511,753k p=0.005 n=6
Parse Time 2.71s (± 0.89%) 2.70s (± 0.89%) ~ 2.67s 2.73s p=0.626 n=6
Bind Time 0.98s (± 0.85%) 0.99s (± 0.76%) ~ 0.98s 1.00s p=0.209 n=6
Check Time 17.25s (± 0.21%) 17.29s (± 0.29%) ~ 17.22s 17.36s p=0.199 n=6
Emit Time 0.00s (± 0.00%) 0.00s (± 0.00%) ~ 0.00s 0.00s p=1.000 n=6
Total Time 20.94s (± 0.25%) 20.98s (± 0.31%) ~ 20.91s 21.08s p=0.377 n=6
mui-docs - node (v18.15.0, x64)
Memory used 1,748,236k (± 0.00%) 1,748,309k (± 0.00%) +73k (+ 0.00%) 1,748,254k 1,748,347k p=0.008 n=6
Parse Time 9.93s (± 0.28%) 9.96s (± 0.52%) ~ 9.90s 10.04s p=0.466 n=6
Bind Time 3.35s (± 0.38%) 3.36s (± 1.05%) ~ 3.32s 3.42s p=0.743 n=6
Check Time 81.95s (± 0.41%) 81.81s (± 0.31%) ~ 81.46s 82.22s p=0.471 n=6
Emit Time 0.20s (± 2.62%) 0.21s (± 6.02%) ~ 0.19s 0.22s p=0.077 n=6
Total Time 95.43s (± 0.34%) 95.34s (± 0.24%) ~ 95.10s 95.74s p=0.575 n=6
self-build-src - node (v18.15.0, x64)
Memory used 2,306,658k (± 0.04%) 2,306,945k (± 0.05%) ~ 2,305,247k 2,308,180k p=0.575 n=6
Parse Time 7.39s (± 0.64%) 7.39s (± 0.81%) ~ 7.34s 7.50s p=1.000 n=6
Bind Time 2.72s (± 0.54%) 2.74s (± 0.63%) +0.02s (+ 0.86%) 2.72s 2.77s p=0.036 n=6
Check Time 49.36s (± 0.69%) 49.20s (± 0.66%) ~ 48.90s 49.80s p=0.298 n=6
Emit Time 3.94s (± 2.42%) 3.88s (± 1.86%) ~ 3.79s 3.96s p=0.230 n=6
Total Time 63.43s (± 0.61%) 63.20s (± 0.65%) ~ 62.82s 63.96s p=0.336 n=6
self-build-src-public-api - node (v18.15.0, x64)
Memory used 2,381,320k (± 0.03%) 2,380,984k (± 0.03%) ~ 2,380,152k 2,381,693k p=0.575 n=6
Parse Time 6.21s (± 0.47%) 6.18s (± 0.87%) ~ 6.11s 6.24s p=0.871 n=6
Bind Time 2.09s (± 1.71%) 2.06s (± 1.02%) ~ 2.02s 2.08s p=0.089 n=6
Check Time 40.23s (± 0.39%) 40.18s (± 0.33%) ~ 39.97s 40.33s p=0.689 n=6
Emit Time 3.24s (± 4.86%) 3.18s (± 1.59%) ~ 3.14s 3.26s p=0.422 n=6
Total Time 51.77s (± 0.59%) 51.61s (± 0.32%) ~ 51.38s 51.82s p=0.230 n=6
self-compiler - node (v18.15.0, x64)
Memory used 419,088k (± 0.01%) 419,134k (± 0.01%) ~ 419,104k 419,172k p=0.066 n=6
Parse Time 4.18s (± 0.47%) 4.21s (± 0.42%) ~ 4.19s 4.24s p=0.054 n=6
Bind Time 1.60s (± 1.27%) 1.60s (± 1.04%) ~ 1.58s 1.62s p=1.000 n=6
Check Time 22.33s (± 0.25%) 22.27s (± 0.49%) ~ 22.16s 22.41s p=0.422 n=6
Emit Time 1.71s (± 1.45%) 1.72s (± 0.68%) ~ 1.70s 1.73s p=0.684 n=6
Total Time 29.83s (± 0.22%) 29.80s (± 0.32%) ~ 29.68s 29.91s p=0.748 n=6
vscode - node (v18.15.0, x64)
Memory used 2,914,035k (± 0.00%) 2,914,128k (± 0.00%) ~ 2,913,976k 2,914,238k p=0.230 n=6
Parse Time 16.46s (± 0.36%) 16.49s (± 0.38%) ~ 16.40s 16.57s p=0.420 n=6
Bind Time 4.94s (± 1.64%) 5.04s (± 2.76%) ~ 4.89s 5.20s p=0.172 n=6
Check Time 87.95s (± 0.39%) 87.84s (± 0.38%) ~ 87.46s 88.25s p=0.810 n=6
Emit Time 23.45s (± 0.60%) 24.34s (± 7.60%) ~ 23.52s 28.11s p=0.065 n=6
Total Time 132.80s (± 0.19%) 133.70s (± 1.46%) ~ 132.65s 137.64s p=0.575 n=6
webpack - node (v18.15.0, x64)
Memory used 409,364k (± 0.01%) 409,467k (± 0.01%) +103k (+ 0.03%) 409,413k 409,530k p=0.020 n=6
Parse Time 3.92s (± 0.83%) 3.91s (± 1.13%) ~ 3.86s 3.98s p=0.868 n=6
Bind Time 1.66s (± 1.19%) 1.66s (± 0.74%) ~ 1.64s 1.67s p=0.742 n=6
Check Time 16.95s (± 0.33%) 16.97s (± 0.43%) ~ 16.89s 17.06s p=0.748 n=6
Emit Time 0.00s (± 0.00%) 0.00s (± 0.00%) ~ 0.00s 0.00s p=1.000 n=6
Total Time 22.53s (± 0.24%) 22.54s (± 0.22%) ~ 22.48s 22.60s p=0.808 n=6
xstate - node (v18.15.0, x64)
Memory used 672,009k (± 0.01%) 672,337k (± 0.01%) +328k (+ 0.05%) 672,236k 672,470k p=0.005 n=6
Parse Time 4.12s (± 0.41%) 4.13s (± 0.66%) ~ 4.09s 4.17s p=0.376 n=6
Bind Time 1.87s (± 1.14%) 1.88s (± 0.99%) ~ 1.86s 1.90s p=0.564 n=6
Check Time 3.44s (± 0.22%) 3.43s (± 0.89%) ~ 3.38s 3.45s p=0.869 n=6
Emit Time 0.03s (± 0.00%) 0.03s (±14.39%) ~ 0.02s 0.03s p=0.405 n=6
Total Time 9.47s (± 0.31%) 9.46s (± 0.52%) ~ 9.40s 9.54s p=0.809 n=6
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • Angular - node (v18.15.0, x64)
  • Compiler-Unions - node (v18.15.0, x64)
  • Monaco - node (v18.15.0, x64)
  • TFS - node (v18.15.0, x64)
  • material-ui - 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)
  • vscode - node (v18.15.0, x64)
  • webpack - node (v18.15.0, x64)
  • xstate - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

Developer Information:

Download Benchmarks

@typescript-bot
Copy link
Collaborator

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

Everything looks good!

@sandersn
Copy link
Member Author

Requires slightly more memory presumably because it adds a few more types and properties.

@jakebailey
Copy link
Member

@typescript-bot run dt

@typescript-bot
Copy link
Collaborator

typescript-bot commented Apr 17, 2024

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

Command Status Results
run dt ✅ Started 👀 Results

@typescript-bot
Copy link
Collaborator

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

There were interesting changes:

Branch only errors:

Package: webrtc
Error:

Error: 
/mnt/vss/_work/1/DefinitelyTyped/types/webrtc/RTCPeerConnection.d.ts
  51:5  error  TypeScript@local compile error: 
All declarations of 'local' must have identical modifiers                                                                                                        @definitelytyped/expect
  51:5  error  TypeScript@local compile error: 
Subsequent property declarations must have the same type.  Property 'local' must be of type 'RTCIceCandidate', but here has type 'RTCIceCandidate | undefined'   @definitelytyped/expect
  52:5  error  TypeScript@local compile error: 
All declarations of 'remote' must have identical modifiers                                                                                                       @definitelytyped/expect
  52:5  error  TypeScript@local compile error: 
Subsequent property declarations must have the same type.  Property 'remote' must be of type 'RTCIceCandidate', but here has type 'RTCIceCandidate | undefined'  @definitelytyped/expect

✖ 4 problems (4 errors, 0 warnings)

    at combineErrorsAndWarnings (/mnt/vss/_work/1/DefinitelyTyped/node_modules/.pnpm/@[email protected][email protected]/node_modules/@definitelytyped/dtslint/dist/index.js:194:28)
    at runTests (/mnt/vss/_work/1/DefinitelyTyped/node_modules/.pnpm/@[email protected][email protected]/node_modules/@definitelytyped/dtslint/dist/index.js:186:20)

You can check the log here.

@jakebailey
Copy link
Member

Ah, webrtc, our old friend

@sandersn
Copy link
Member Author

Yeah, that's expected, since types/webrtc is supposed to be a superset of whatever's in the DOM.

I'll merge this tomorrow after I merge all the test baseline conflicts. Is this the result of the new performance stats? Or maybe it's the generated-type spans.

@jakebailey
Copy link
Member

Is this the result of the new performance stats? Or maybe it's the generated-type spans.

Both, probably.

@sandersn sandersn merged commit e04a27c into main Apr 19, 2024
25 checks passed
@sandersn sandersn deleted the dom-update-2024-04-16 branch April 19, 2024 15:52
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
None yet
Development

Successfully merging this pull request may close these issues.

3 participants