Skip to content

feat(馃敽): new async handling - #275

Merged
wcandillon merged 27 commits into
mainfrom
new-async
Oct 28, 2025
Merged

wcandillon merged 27 commits into
mainfrom
new-async

Conversation

@wcandillon

@wcandillon wcandillon commented Oct 27, 2025 •

Copy link
Copy Markdown
Owner
  • Make the module fabric only for better symmetric with the Web (sync access to layout size)
  • Proper async model for WebGPU
  • on iOS run the configure method on the UI thread.

fixes #247
fixes #243
fixes #179

@wcandillon

wcandillon commented Oct 27, 2025 •

Copy link
Copy Markdown
Owner Author

@hmallen99 you should be interested by this rewrite. We are no "properly" using WGPU async model.

@wcandillon

Copy link
Copy Markdown
Owner Author

@iwoplaza @mhawryluk this will fix the issue you noticed with the effect hooks being unstable. Now it works exactly like on the web, no less, no more. At this point the only difference I can think of with the web is the ctx.present() needed. But we might think about that.

This also make hot reloading work properly now.

@wcandillon
wcandillon requested a review from piaskowyk October 27, 2025 14:31
@wcandillon

Copy link
Copy Markdown
Owner Author

@piaskowyk it looks like outside the JS thread we need to use setImmediate() (queueMicroTask won't work?)

@wcandillon
wcandillon merged commit b326ac7 into main Oct 28, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant