-
Notifications
You must be signed in to change notification settings - Fork 935
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
Setup Assistant 2 #894
Comments
I'm not aware of the details of the GSoC assignment, so this is just a thought and may be already covered by some other requirement / idea (and something I've played with in the past): it might be nice to make the SA plugin based, where the overal UI and IO framework is provided by the SA, but plugins contribute the actual screens of the wizard. My primary motivation for that was (now describing a situation in the past) a use-case we had in a project where we extended MoveIt with specific functionality that the SA had no way of configuring for. Instead of forking, a plugin-based approach allowed to provide just a single screen in a One example is a SA screen that uses an interface to a particular industrial robot controller and shows a UI to import all 'home positions' defined on that controller as named states in a MoveIt configuration by clicking a button. |
To make the SA plugin based does actually sounds like a very nice idea. What do you think? |
We are about to start working on our own "mikado_setup_assistant" (name not final) for our software, sharing plugins either way sounds like a great idea! (I.e. having a plugin system should hopefully also allow loading these in a different host application) |
@MohmadAyman I would say the plugins should be a lower priority than the other features we are focusing on, but if there was time at the end of the summer then of course it sounds great |
Hi @simonschmeisser , have you guys started working on your "mikado_setup_assistant"? |
@MohmadAyman no, not yet. I cannot really give a prediction yet when we will either |
@MohmadAyman please update this issue with your final updates, then we should consider closing it |
Done, Can we leave it until the tutorials have been updated? so they don't get forgotten. |
I believe this can now be closed. Congrats @MohmadAyman for a very successful GSoC project! Your continued involvement in MoveIt! including code review and code contributions are always welcomed 💯 |
I've opened a PR for the Plugin based SA #1243 , would be nice to have your input. |
Description
Hello there, I'm Mohamad Ayman a Computer Engineering Masters student and a Google Summer of Code student.
My proposal this summer focuses on improving the Setup Assistant by automating the generation of some configurations and adding more GUIs to make the generation of some easier and more straight forward, along with improving the integration with Gazebo where we one wont have to add configs or edit the robot file manually for it to work with gazebo.
Below are the description of the improvements broken into tasks, I will update the tasks each time a related PR is opened, any suggestions or feedback are highly appreciated.
Project description on Google Summer of Code website.
New screen for automatically generating interfaces to low level controllers
PR Setup assistant automating generation of controllers configspull request Read and write ROS controllers yaml file #951pull request ROS Controllers screen added to SA #994Improving gazebo integration.
pull request Simulation Screen for the SA #956pull request Added gazebo launch file #936Perception screen for using laser scanner point clouds.
pull request Perception screen added for the SA #969New grasping screen for setting up end effectors.
[Blocked for now because the backend is still not ready]
Parse existing text files [launch/config/package] and adding more GUIs for loading and saving settings.
[Basically each of the previous points contribute to this point]
others
SA Navigation hot fix #1022Planning Groups screen re-configured #1017fix bug in perception screen #1013Renamed variable names in setup assistant widget to follow naming convention #995remove unused variable in moveit_config_data -> setup assistant #924Fix all references to "MoveIt" to include exclamation mark e.g. "MoveIt!" #1020The text was updated successfully, but these errors were encountered: