Incorrect WARN when using sub states about bevy HOT 5 CLOSED

Oh, that's an interesting issue.
So in_state uses the existence of State<S> to decide whether a state "exists", but computed/sub states work on optional resources.

We can either use a different resource to verify that a state is installed (e.g. Events<StateTransitionEvent<S>> is something I use), but then we need to add it as an argument to in_state 😛

Alternatively we can just remove the warning, it warns about something pretty obvious

from bevy.

@alice-i-cecile - can you mark me as assigned here? I'm working on a solution RN

from bevy.

FYI @MiniaczQ @lee-orr

from bevy.

Oh, that's an interesting issue. So in_state uses the existence of State<S> to decide whether a state "exists", but computed/sub states work on optional resources.

We can either use a different resource to verify that a state is installed (e.g. Events<StateTransitionEvent<S>> is something I use), but then we need to add it as an argument to in_state 😛

Alternatively we can just remove the warning, it warns about something pretty obvious

The warning makes sense for States because it is assumed that they are initialised (idk how appropriate this assumption is). But it doesn't make sense for sub-states as they are inherently optional.

As an outsider, I will cautiously make the suggestion that sub states can be generalised so that all states are sub states. I don't think there needs to be two classes of states in this way (which makes the interface more complex). A regular state could then just be a sub state of some built-in unary root state.

Then if add_sub_state was not called for a state, then any in_state calls for that would only return false which is how sub-states behave.

from bevy.

The warning makes sense for States because it is assumed that they are initialised (idk how appropriate this assumption is). But it doesn't make sense for sub-states as they are inherently optional.

I'm uncertain what you mean by "it is assumed that they are initialized"? Can you elaborate?
I think my confusion is that all state types need to be registered with the app to work, so "initialized" in that sense applies to everything - but if you mean they can or can't be None that's a different distinction.

from bevy.