Conversation
this is the first thing that claude spit out, probably will need revisiting
|
|
Great job! No new security vulnerabilities introduced in this pull requestUse @Checkmarx to interact with Checkmarx PR Assistant. |
it actually kinda doesn't make sense to have it on the top level of the library, since you need some kind of a reference to the renderer to get the current state (like camera position)
the setup script should detect the version based on project dependencies
| steps: | ||
| - uses: actions/checkout@v5 | ||
| - uses: biomejs/setup-biome@v2 | ||
| with: |
There was a problem hiding this comment.
By deleting the fixed version specifier, the setup script will look at what version is declared as dependency in the project

This PR adds capability to take higher resolution screenshots.
Previously the users were expected to simply make the canvas as large as possible for larger screenshots. This is obviously limited by the physical size of their display. The new feature uses an OffscreenCanvas to render the scene at an arbitrary size.
The canvas background is kept transparent, making it easier to manipulate the images in post-production.
Usage:
TODOs:
qualityparam > 1: the SSAO visuals change significantly