-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update Bevy to 0.15, update Rerun to 0.20 #4
Conversation
arrow length shenanigans related to |
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.
Amazing, thanks so much 🙇 (I'll take care of my comments)
@@ -19,11 +19,10 @@ https://github.com/rerun-io/revy/assets/2910679/cd096cbe-5e68-4acf-8010-e6c32c55 | |||
|
|||
## Examples | |||
|
|||
|[Breakout](https://github.com/bevyengine/bevy/blob/v0.13.0/examples/games/breakout.rs)|[3D shapes](https://github.com/bevyengine/bevy/blob/v0.13.0/examples/3d/3d_shapes.rs)|[Alien Cake Addict](https://github.com/bevyengine/bevy/blob/v0.13.0/examples/games/alien_cake_addict.rs)| | |||
|[Breakout](https://github.com/bevyengine/bevy/blob/v0.15.0/examples/games/breakout.rs)|[3D shapes](https://github.com/bevyengine/bevy/blob/v0.15.0/examples/3d/3d_shapes.rs)|[Alien Cake Addict](https://github.com/bevyengine/bevy/blob/v0.15.0/examples/games/alien_cake_addict.rs)| |
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.
Once this is merged, I'll make a release and link to our examples instead, so people can see what the modification actually looks like.
@@ -254,7 +233,7 @@ impl<C: rerun::LoggableBatch> rerun::ComponentBatch for Aliased<C> {} | |||
|
|||
/// Helper to merge any number of [`rerun::AsComponents`]. | |||
#[allow(dead_code)] | |||
pub struct ManyAsComponents(Vec<Box<dyn rerun::AsComponents>>); | |||
pub struct ManyAsComponents(pub Vec<Box<dyn rerun::AsComponents>>); |
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.
Ooooh, I should add the blanket impls for this
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.
Considered adding blueprints to the example links, but wasn't all that interesting (maybe hide panels, but not much more to be done here)
Performance overall comparable, Rerun 0.20 is generally a little bit faster, but not much here.
What's really annoying though is the invasion of transform arrows: had to turn them off everywhere in a hacky way otherwise things get way to overloaded. Will follow-up over on Rerun about that.
Updated readme (with updated examples) rendered:
https://github.com/rerun-io/revy/blob/update-bevy-and-rerun/README.md