-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
[Merged by Bors] - EntityRenderCommand and PhaseItemRenderCommand #3111
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The EntityRenderCommand
, EntityPhaseItem
and CachedPipelinePhaseItem
traits look great. I will try them soon.
I haven't reviewed the custom_shader part.
It seems that the PR title needs updating, there are no PhaseItemRenderCommand
s
Oops good call @Davier! Updated to include the proper trait names. |
33a26a7
to
64239ea
Compare
(just renamed the commit ... no changes) |
bors r+ |
Adds new `EntityRenderCommand`, `EntityPhaseItem`, and `CachedPipelinePhaseItem` traits to make it possible to reuse RenderCommands across phases. This should be helpful for features like #3072 . It also makes the trait impls slightly less generic-ey in the common cases. This also fixes the custom shader examples to account for the recent Frustum Culling and MSAA changes (the UX for these things will be improved later).
Pull request successfully merged into pipelined-rendering. Build succeeded: |
Adds new
EntityRenderCommand
,EntityPhaseItem
, andCachedPipelinePhaseItem
traits to make it possible to reuse RenderCommands across phases. This should be helpful for features like #3072 . It also makes the trait impls slightly less generic-ey in the common cases.This also fixes the custom shader examples to account for the recent Frustum Culling and MSAA changes (the UX for these things will be improved later).