Repository navigation
Add typography benchmark tests for textToContours() - #9268
LalitNarayanYadav wants to merge 3 commits into
Conversation
| ); | ||
| } | ||
|
|
||
| for (let testCase of TO_CONTOURS_CASES) { |
There was a problem hiding this comment.
For consistency, we might want to sequence textToContours() tests after textToPoints() in all 3 renderers
There was a problem hiding this comment.
Done in all 3 renderers. The order is now points, then contours, then model.
| {label: "textToModel() paragraph", str: strs.paragraph, textSize: 20, sampleFactor: 0.5, render: false}, | ||
| ]; | ||
|
|
||
| const TO_CONTOURS_CASES = TO_POINTS_CASES.map((c) => ({ |
There was a problem hiding this comment.
Similar note RE: sequencing these after TO_POINTS_CASES
There was a problem hiding this comment.
Done, TO_CONTOURS_CASES now sits right after TO_POINTS_CASES
| ]; | ||
|
|
||
| const TO_CONTOURS_CASES = TO_POINTS_CASES.map((c) => ({ | ||
| ...c, |
There was a problem hiding this comment.
I appreciate the test case reuse here. How might we also test the expected contour count?
There was a problem hiding this comment.
Added an EXPECTED_CONTOURS lookup, indexed like TO_POINTS_CASES, and an assert.equal on contours.length in all 3 renderers. The counts are 16 for the single word, 76 for 10 words and 1167 for the paragraph, and they’re the same in 2D, WebGL and WebGPU
Resolves #8968
Changes:
textToContours()benchmarks totest/bench/typography.bench.js, continuing Add typography performance tests #8918TO_CONTOURS_CASESis derived fromTO_POINTS_CASES, so the strings, text sizes andsampleFactorare identical across thetextTo___benchmarksdrawContours()helper for therender: truecaseTO_CONTOURS_CASESand the contours benchmarks right after thetextToPoints()ones in all 3 rendererstextToPoints()expectations (samesampleFactor: 0.5, same variance)EXPECTED_CONTOURS(16 for the single word, 76 for 10 words, 1167 for the paragraph)Testing:
npx vitest bench --run test/bench/typography.bench.js --project unit-tests-webgpu -t "textToContours". Contour counts and point totals pass in 2D, WebGL and WebGPU.textToPoints()andtextToModel()ones, which record no samples, withCannot read properties of undefined (reading 'createShaderModule')raised fromdrawContours()in the contours case. This looks like no WebGPU device being available in headless Chromium here, so those cases aren't verified locally. It would be great if someone could confirm them in CI or on another setup.Note: in WebGL, the render case is about 10x slower than the non-render case, because
drawContours()calls beginShape/endShape once per contour.cc @shakeelmohamed for review.
Screenshots of the change:
N/A (benchmark tests only)
PR Checklist
npm run lintpasses