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

[Merged by Bors] - bevy_render: Support removal of nodes, edges, subgraphs #3048

Closed
wants to merge 4 commits into from

Conversation

superdump
Copy link
Contributor

@superdump superdump commented Oct 29, 2021

Add support for removing nodes, edges, and subgraphs. This enables live re-wiring of the render graph.

This was something I did to support the MSAA implementation, but it turned out to be unnecessary there. However, it is still useful so here it is in its own PR.

@inodentry inodentry added A-Rendering Drawing game state to the screen C-Enhancement A new feature C-Usability A simple quality-of-life change that makes Bevy easier to use S-Needs-Review labels Oct 31, 2021
@superdump superdump changed the base branch from pipelined-rendering to main January 15, 2022 23:53
@superdump superdump changed the title bevy_render2: Support removal of nodes, edges, subgraphs bevy_render: Support removal of nodes, edges, subgraphs Jan 15, 2022
@superdump
Copy link
Contributor Author

This is now updated on top of main. I've also addressed the feedback to get it into a better state. However, I haven't tested it other than to check it builds and passes cargo run -p ci. I suppose it should have a bunch of unit tests.

@superdump
Copy link
Contributor Author

Also, when I changed the PR base, it looks like CI broke?

@superdump
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request Mar 20, 2022
Copy link
Contributor

@jakobhellermann jakobhellermann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, I left some small suggestions but other than that it should be good to merge

crates/bevy_render/src/render_graph/graph.rs Outdated Show resolved Hide resolved
crates/bevy_render/src/render_graph/graph.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@inodentry inodentry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I got no more feedback to add, besides the minor things from the previous review.

@inodentry inodentry added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Mar 21, 2022
Copy link
Member

@cart cart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally looks good to me! We should clean up one "legacy correctness issue" that existed prior to these changes. I'll do that now so we can merge this asap.

crates/bevy_render/src/render_graph/graph.rs Show resolved Hide resolved
@cart
Copy link
Member

cart commented Mar 21, 2022

bors r+

bors bot pushed a commit that referenced this pull request Mar 21, 2022
Add support for removing nodes, edges, and subgraphs. This enables live re-wiring of the render graph.

This was something I did to support the MSAA implementation, but it turned out to be unnecessary there. However, it is still useful so here it is in its own PR.

Co-authored-by: Carter Anderson <mcanders1@gmail.com>
@bors bors bot changed the title bevy_render: Support removal of nodes, edges, subgraphs [Merged by Bors] - bevy_render: Support removal of nodes, edges, subgraphs Mar 22, 2022
@bors bors bot closed this Mar 22, 2022
aevyrie pushed a commit to aevyrie/bevy that referenced this pull request Jun 7, 2022
)

Add support for removing nodes, edges, and subgraphs. This enables live re-wiring of the render graph.

This was something I did to support the MSAA implementation, but it turned out to be unnecessary there. However, it is still useful so here it is in its own PR.

Co-authored-by: Carter Anderson <mcanders1@gmail.com>
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
)

Add support for removing nodes, edges, and subgraphs. This enables live re-wiring of the render graph.

This was something I did to support the MSAA implementation, but it turned out to be unnecessary there. However, it is still useful so here it is in its own PR.

Co-authored-by: Carter Anderson <mcanders1@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Rendering Drawing game state to the screen C-Enhancement A new feature C-Usability A simple quality-of-life change that makes Bevy easier to use S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants