Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for one-time takeoff/landing #2

Open
christophebedard opened this issue Apr 28, 2018 · 2 comments
Open

Add support for one-time takeoff/landing #2

christophebedard opened this issue Apr 28, 2018 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@christophebedard
Copy link
Member

Puisque le takeoff et le landing ne sont effectués qu'une seule fois chacun (au début/à la fin), il faut pouvoir débuter absolument par le takeoff, poursuivre avec le reste, et finir (en espérant) avec le landing.

L'option que j'envisage :

Un genre de init() appelé au tout début de la Strategy : le premier Behaviour est alors un genre de TakeoffBehaviour qui gère la TakeoffCommand. Lorsque le takeoff est terminé, on poursuit avec la boucle update() infinie pour le switch d'autres Behaviour. Le landing serait appelé par une méthode land() de Strategy, qui changerait pour un genre de LandingBehaviour/Command.

Aussi, comme failsafe additionnel, land() pourrait être appelée aussi après la boucle ros::ok() et/ou dans le destructeur de Strategy (mais je sais pas trop si c'est une bonne idée), au cas où ROS plante.

Ce qui me tracasse, c'est que cette option fait en sorte que Strategy gère (presque) directement la commande de takeoff/landing, ce qui casse un peu le principe strategy/behaviour/command. Toutefois, Strategy est le seul objet qui ne va pas être remplacé par un autre objet (ce que Behaviour et Command font), alors ce serait une place facile où mettre le "flag" de takeoff/landing.

@lajoiepy conseils/commentaires?

@christophebedard christophebedard added the enhancement New feature or request label Apr 28, 2018
@christophebedard christophebedard self-assigned this Apr 28, 2018
@lajoiepy
Copy link
Member

lajoiepy commented Apr 29, 2018 via email

@christophebedard
Copy link
Member Author

hmmm I guess que non 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants