idanarye / bevy-yoetz

Apache License 2.0
7 stars 2 forks source link

Move `act` systems into `Update` together with `input` systems. #7

Closed seivan closed 1 month ago

seivan commented 1 month ago

Also moved input systems. Doesn't need lerp for now.

idanarye commented 1 month ago

Please explain? Why shouldn't they be in the FixedUpdate schedule? You've argued in #3 that behavioral systems should use fixed timesteps.

seivan commented 1 month ago

Please explain? Why shouldn't they be in the FixedUpdate schedule? You've argued in #3 that behavioral systems should use fixed timesteps.

Because you're mutating rendered objects directly in the system. That's why you'd need lerp if you'd had a slower tick rate for FixedUpdate, but I am guessing the default one matches the frame rate, it's not really visible.