Closed marotili closed 5 years ago
This PR looks good to me.
However, if we want to move the load_factions
to a separate LoadingState
, we're gonna hit a problem. The WriteStorage<FactionEnemies>
is not setup yet at that stage since it is setup with the main game state dispatcher.
@sunreef I tested this and it works because PrefabLoaderSystem::<combat::FactionPrefabData>
is declared in the game state dispatcher. The prefab loader system has WriteStorage<FactionEnemies>
in its system data.
This should allow us to be a little bit more flexible. In the future, we are going to replace the concept of factions with a more continuous data model. Our simulation should not be black and white.