Invalidated Entity's

[zicklag]

Invalidated Entity's

[gschup]

You raise a valid concern. I did not think about hierarchical entities when writing this plugin. I have not tried it, but I doubt it would work.

[gschup]

I will take a deeper look into this on the weekend. The more I learn about bevy, the more I am certain that this version of the bevy_ggrs plugin is a prototpye that needs a rewrite :D

[zicklag]

Yeah, I'll be thinking about this too.

I like that this plugin isn't super complicated anyway. I did a little bit of looking at the ggrs crate API and it's pretty simple, too. Good job.

---

While brainstorming I was wondering if we could have a sub-world that we do the rollback system in, which would have it's own entities, and could we somehow snapshot the entire world, including it's entities, and then have like an extract phase similar to how is used for Bevy rendering to pull the needed components into the normal Bevy world before rendering.

I don't actually think that would work with the current state of Bevy, but it's food for thought. I'm still thinking... :D

[zicklag]

Something maybe to look into is Bevy's DynamicScene and also the EntityMap used to write DynamicScenes to the world.

[zicklag]

Aha! This code is key to how Bevy imports hierarchy's from scene files:

        for registration in type_registry.iter() {
            if let Some(map_entities_reflect) = registration.data::<ReflectMapEntities>() {
                map_entities_reflect
                    .map_entities(world, entity_map)
                    .unwrap();
            }
        }

It uses ReflectMapEntities to update all the Parent and Children components with the mapped entities.

I'm still fitting this together in my head, but I think we can make this work with that.

[zicklag]

I just found that invalidating entities is still an issue, but luckily I also discovered the fix we need in the bevy_hierarchy crate. Just posting here for a heads-up in case anybody runs into it:

bevyengine/bevy#6790

For now, in my game I've just patched Bevy with the PR linked in the issue above to fix it.