uzh-rpg / rpg_quadrotor_control

Quadrotor control framework developed by the Robotics and Perception Group
Other
585 stars 185 forks source link

GUI updates #56

Closed kohlerj closed 4 years ago

kohlerj commented 6 years ago

I'm starting to rearrange the widgets.

kohlerj commented 6 years ago

@mfaessle Can you just give me your impression. Ideally I want to make the GUI sort of failure proof in that the button are enabled only if the state machine allows it. That would require a state machine in the GUI as well but if you are interested, why not!?

foehnx commented 6 years ago

Instead of a state machine in the GUI can't we just do some safety checks for each button? so that each button checks whether it can be enabled or not?

mfaessle commented 6 years ago

So technically you have the autopilot's state available and could enable or disable buttons by replicating what is allowed in the autopilot. Safety checks should not be in the gui and are also not required, the autopilot does all this. Disabling buttons would only be visual feedback for the user that he can currently not perform certain actions. But he will also figure this out if he tries an action and the autopilot just doesn't do it. So I think this is a complete overbite feature