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

[ROS_MELODIC] Add Mellinger's and Silano et. al controllers #29

Merged
merged 21 commits into from
Apr 22, 2020

Conversation

gsilano
Copy link
Owner

@gsilano gsilano commented Feb 2, 2020

The new version aims to add two news controller algorithms: the Mellinger [1] and Silano et. al [2]. This branch concerns ROS Melodic version.

[1] D. Mellinger and V. Kumar, "Minimum snap trajectory generation and control for quadrotors," 2011 IEEE International Conference on Robotics and Automation, Shanghai, 2011, pp. 2520-2525. DOI: 10.1109/ICRA.2011.5980409
[2] G. Silano, P. Oppido and L. Iannelli, "Software-in-the-loop simulation for improving flight control system design: a quadrotor case study," 2019 IEEE International Conference on Systems, Man and Cybernetics (SMC), Bari, Italy, 2019, pp. 466-471. DOI: 10.1109/SMC.2019.8914154

@gsilano gsilano self-assigned this Feb 2, 2020
@gsilano gsilano added the type: enhancement New feature or request label Feb 2, 2020
rotors_gazebo/CHANGELOG.rst Outdated Show resolved Hide resolved
@gsilano gsilano mentioned this pull request Feb 23, 2020
@gsilano
Copy link
Owner Author

gsilano commented Mar 9, 2020

@marco-marino95, please open an issue related to your problem. This conversation concerns the PR that has not yet been integrated into the master.

@gsilano gsilano changed the title [NOT_MERGE_YET] [ROS_MELODIC] Add Mellinger's and Silano et. all controllers [NOT_MERGE_YET] [ROS_MELODIC] Add Mellinger's and Silano et. al controllers Apr 5, 2020
gsilano added a commit to gsilano/mav_comm that referenced this pull request Apr 14, 2020
@gsilano gsilano marked this pull request as draft April 14, 2020 17:43
@senceryazici
Copy link

Hi,
Are there any estimations when this Pull Request will be ready to merge ?

Thanks

@gsilano
Copy link
Owner Author

gsilano commented Apr 15, 2020

Hi @senceryazici! No, sorry. I'm still working on it and I don't know how long it will take.

Copy link
Owner Author

@gsilano gsilano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The crazyflie_internal_model_controller.yaml file contains not working control gains. As with BebopS, it is needed to run an optimization script to find a working set of control gains. Instead, a complete review is required for the Mellinger's controller implementation.

gsilano added a commit that referenced this pull request Apr 16, 2020
@gsilano gsilano changed the title [NOT_MERGE_YET] [ROS_MELODIC] Add Mellinger's and Silano et. al controllers [ROS_MELODIC] Add Mellinger's and Silano et. al controllers Apr 22, 2020
@gsilano gsilano marked this pull request as ready for review April 22, 2020 16:35
@gsilano gsilano merged commit 0251491 into dev/ros-melodic Apr 22, 2020
@gsilano gsilano deleted the feature/controller-ros-melodic branch April 22, 2020 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants