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

Overhaul renderer structure #2

Merged
merged 13 commits into from
Jan 21, 2024
Merged

Overhaul renderer structure #2

merged 13 commits into from
Jan 21, 2024

Conversation

ten3roberts
Copy link
Owner

@ten3roberts ten3roberts commented Jan 13, 2024

Makes the renderer's structure consistent.

  • Introduces draw_shape which influences the renderer a widget will employ, rather than having it inferred by its components, leading to possible dual-rendering and overlap.
  • Makes the renderer own the resources, and move the Wgpu resources out of the ECS
  • Improves object uniform updating
  • Composable renderer system

@ten3roberts ten3roberts changed the title Overhaul rendererer structure Overhaul renderer structure Jan 13, 2024
Makes the renderer structure consistent.

- Introduces `draw_shape` which renderer a widget will employ, rather
  than being infered by its component, leading to possible
  dual-rendering and overlap.
- Makes the renderer own the resources, and move the Wgpu resources out
  of the ECS
- Improves object uniform updating
- Composable renderer system
@ten3roberts ten3roberts force-pushed the feat-renderer-overhaul branch from aa3d234 to 855d4a0 Compare January 13, 2024 11:50
Uses a store and handle based approach for storing renderer resources
such as shaders and bind groups, rather than storing them in the world
behind Arcs
@ten3roberts ten3roberts force-pushed the feat-renderer-overhaul branch from c8d770a to 24dade8 Compare January 15, 2024 23:21
@ten3roberts ten3roberts merged commit 240531d into main Jan 21, 2024
8 checks passed
@ten3roberts ten3roberts deleted the feat-renderer-overhaul branch January 21, 2024 22:58
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