Skip to content

Add typography to model performance tests #8993

Open
michtesar wants to merge 7 commits into
processing:mainfrom
michtesar:8970-bench-text-to-model
Open

Add typography to model performance tests #8993
michtesar wants to merge 7 commits into
processing:mainfrom
michtesar:8970-bench-text-to-model

Conversation

@michtesar

Copy link
Copy Markdown

Resolves #8970

Changes:
This PR continues the typography benchmark work from #8918 (branched from #8918).

It adds TO_MODEL_CASES using the same strings, text sizes, sample factors, and rendering configurations as the existing textToPoints() benchmarks. The new cases run with both WebGL and WebGPU.

The WebGPU renderer addon is registered explicitly because src/app does not include it. Without registration, the benchmark emits renderer errors and produces no valid textToModel() results.

Benchmarks also verifies that the generated model contains vertices with asserts. This helps detect cases where the benchmark appears to run successfully without producing usable geometry (which it happened before I registered the WebGPU renderer).

Screenshots of the change:

Snímek obrazovky 2026-07-19 v 22 05 53

New tests are running ok

PR Checklist

@michtesar

Copy link
Copy Markdown
Author

Hi @shakeelmohamed @ksen0 here is a PR for the #8970.

I branch off the #8918 and once you are fine with this PR, I will leave a comment about one too there.

Few notes about the changes:

  • I had to add WebGPU registration as without vertices list was empty - tests we passing green but silently failing with errors
  • I only implemented the v2 tests as v1 was not mentioned in the issues, but can add them too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add typography benchmark tests for textToModel()

2 participants