Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: use weak ref slab #135

Merged
merged 5 commits into from
Oct 4, 2024
Merged

fix: use weak ref slab #135

merged 5 commits into from
Oct 4, 2024

Conversation

schell
Copy link
Owner

@schell schell commented Oct 4, 2024

This updates SlabAllocator to use weak references instead of cloning Arcs.

Does the same for Stage and Atlas, as each need to keep track of certain resources in userland.

For Atlas, the repacking method was simplified and improved. Now upkeep is run on every repacking.

Simplifies draw calls by introducing a new type DrawCalls, that abstracts over indirect and direct draw strategies.

Compute culling is no longer configurable. Stage will use indirect drawing+compute culling on supported platforms (everywhere but web).

@schell schell marked this pull request as ready for review October 4, 2024 18:04
@schell schell merged commit b237d42 into main Oct 4, 2024
6 checks passed
@schell schell deleted the fix/use-weak-ref-slab branch October 4, 2024 21:05
schell added a commit that referenced this pull request Dec 27, 2024
* chore: bumped rust-gpu

* consolidate drawing into new draw module, use weak refs for renderlets internally

* fixes after weak ref work

* revert back to spirv-builder v0.9.1 and regen shaders

* track rust-gpu main
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.

1 participant