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

generalize the motionautomaton to work with a configuration file. #3

Open
ritwika314 opened this issue May 29, 2019 · 1 comment
Open
Assignees
Labels
enhancement New feature or request

Comments

@ritwika314
Copy link
Contributor

The motionautomaton is customized to the vicon , vicon topics and cyphyhouse robots. Make a configuration file where any group can add their localization, and bots. Maybe, create an abstract class with override methods.

@hc825b hc825b added the enhancement New feature or request label May 31, 2019
@hc825b
Copy link
Member

hc825b commented May 31, 2019

We can change MotionAutomaton class to be an abstract class with only public methods (like interface in Java). Current code will be an implementation specifically using ROS and Vicon.
This implementation may be moved to the hardware controller component instead of middle-ware. This also removes the dependency on ROS for middle-ware.

There will be some changes on Gvh to initialize with an instance of MotionAutomaton. And choosing between implementations will be at, for example, basicFollowApp.py.

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

3 participants