bevyengine / bevyengine/bevy

Deferred Rendering follow up items

Open
#10,101 3 comments 0 reactions 0 assignees View on GitHub
A-Rendering C-Feature C-Tracking-Issue
Dominant language
Rust
Stars
48.2k
Forks
4.8k
Avg merge
3d 22h
Merged PRs (30d)
161

Description

### Due to the size of the Deferred Rendering PR it was decided to address some issues in follow PRs. This issue exists to track the progress of the issues identified in the PR code review

- [ ] Make mesh pipeline code reusable https://github.com/bevyengine/bevy/pull/9258#discussion_r1301514675
- [x] Use coordinate transformations from https://github.com/bevyengine/bevy/pull/9726 once it's merged
- [ ] Don't unnecessarily recreate bindings/textures every frame https://github.com/bevyengine/bevy/pull/9258#discussion_r1342191472 https://github.com/bevyengine/bevy/pull/9258#discussion_r1342191971 https://github.com/bevyengine/bevy/pull/9258#discussion_r1349672439
- [x] Automatically determine where/when the color attachment should be cleared: https://github.com/bevyengine/bevy/pull/9258#discussion_r1353824381
- [x] Improve material UX to more cleanly support deferred+forward materials: https://github.com/bevyengine/bevy/pull/9258#pullrequestreview-1669400495 https://github.com/bevyengine/bevy/pull/9258#pullrequestreview-1672742236
- [ ] Copy normals to prepass texture in subsequent pass to minimize deferred attachment size: https://github.com/bevyengine/bevy/pull/9258#discussion_r1357539129

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.