jakobhellermann / bevy-inspector-egui

Inspector plugin for the bevy game engine
Apache License 2.0
1.09k stars 163 forks source link

Inspector systems are ambiguous with the entire Update schedule #169

Open nfagerlund opened 7 months ago

nfagerlund commented 7 months ago

Since inspector_ui<T> and world_inspector_ui take a &mut World and go in the Update schedule, Bevy's ambiguity_detection reporting shows them as having conflicting data access and indeterminate ordering with basically everything. So, your legit determinism problems get drowned out by pairs like these:

 -- world_inspector_ui and mobile_fixed_velocity (in set MovePlanners)
    conflict on: bevy_ecs::world::World
 -- world_inspector_ui and launch_and_fall (in set MovePlanners)
    conflict on: bevy_ecs::world::World
 -- world_inspector_ui and mobile_chase_entity (in set MovePlanners)
    conflict on: bevy_ecs::world::World

Thoughts:

AlexAegis commented 2 months ago

Not using Update would be really helpful for system stepping too. Currently the inspector gui flashes as I step through my systems.