-
-
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
Retain Rendering World #14449
Retain Rendering World #14449
Changes from 33 commits
95b7f0b
17f2bb0
35fe01e
f9bae87
34853b2
000cef4
2255b21
f90c360
756b082
5b5ef76
e15e3a3
e4e0bc0
33cae8d
4a58b0e
3fc2207
f9c8d5a
44d36a4
9a7d641
7989d21
9485455
bb3289b
39e4f2e
4b6506e
03a6e09
b8b8225
24212f5
2de636a
1fbb76c
c25d560
457e83f
16c1093
cba1183
2821849
a86dd39
8422a48
dc5f10f
2a6df8d
ce2ea36
c1a4fb9
672778d
0eed037
5a83da4
ed296f1
bb8774d
94be0cd
560ce59
ab22138
fd36379
4bdfd60
6214e6d
67e7a07
1507437
e5254be
d9ecccd
faf5ef2
f127719
c7ba466
23a8bba
37944a8
1e8601f
7167cc0
bfad1de
8d2c81a
d857e6e
9c1a523
753d286
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -93,6 +93,7 @@ use bevy_render::{ | |
ShaderType, VertexAttribute, VertexBufferLayout, VertexFormat, VertexStepMode, | ||
}, | ||
renderer::RenderDevice, | ||
world_sync::RenderFlyEntity, | ||
Extract, ExtractSchedule, Render, RenderApp, RenderSet, | ||
}; | ||
use bevy_time::Fixed; | ||
|
@@ -432,6 +433,7 @@ fn extract_gizmo_data( | |
}, | ||
(*handle).clone_weak(), | ||
GizmoMeshConfig::from(config), | ||
RenderFlyEntity, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We discussed this before and it still might be the best option for now, but it feels not great to require users to remember this while extracting. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should definitely be renamed before merging. My suggestion would be |
||
)); | ||
} | ||
} | ||
|
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.
Lets see if I understand this correctly: this could be just get_mut() here instead of get_or_spawn(), as the entity should already exist as camera bundles have SyncRenderWorld, correct?
(That said I do think we should leave it as get_or_spawn() in case users aren't using the bundle, this comment is just to see if I understand this PR correctly or not)