perf(browser): improve the parallelisation logic

sheremet-va commented 3 weeks ago

Description

Just experimenting with different methods (this doesn't work yet). Ideally, tinypool should expose a way to do that @AriPerkkio

Related #6834

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
[ ] Ideally, include a test that fails without this PR but passes with it.
[ ] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

[ ] Run the tests with pnpm test:ci.

Documentation

[ ] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

[ ] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

netlify[bot] commented 3 weeks ago

Deploy Preview for vitest-dev ready!

Name	Link
Latest commit	d16306beed731082b649a99161a5db92a7a6b3c0
Latest deploy log	https://app.netlify.com/sites/vitest-dev/deploys/6729fb005e8ecf0008eb840b
Deploy Preview	https://deploy-preview-6856--vitest-dev.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

AriPerkkio commented 3 weeks ago

The support from Tinypool's side is quite complex as Piscina's codebase is not flexible. There's some work done on https://github.com/tinylibs/tinypool/pull/98 but I'm not sure when I'm ready to continue that 😱

vitest-dev / vitest