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

Restructuring of the repository for better overview. #232

Merged
merged 13 commits into from
Mar 8, 2023

Conversation

destogl
Copy link
Member

@destogl destogl commented Feb 2, 2023

This PR introduces the restructured version of the repository.

Each example should be submitted as separate PR to this branch.
All examples should be merged at once.

Resolves #231.

Before merging this PR we should at least do the following:

  • rename use_fake_hardware to use_mock_hardware
  • Remove rrbot_base.launch.py from example folders, i.e., rename it to example launch file and setup parameters needed for the example.
  • Cleanup / revise xacro parameters #240
  • remove use of NOTE: from .rst files and use .. note:: block
  • Use CMakeLists.txt structure for overriding showed in - ros_controls/ros2_control#926
  • Remove test_nodes package from this repository and use ros2_controllers_test_nodes package
  • Update to new ros2controlcli interface #75
  • Check that the .rst files are integrated into documentation on control.ros.org
  • Missing dependencies of examples  #244

Make sure the example implements add details as: (copy this list the PR) - replaces #119

  • Has correct folder structure (Described in the main README.md)
  • Example has "README.rst" with description
  • There are detailed commands how to run an example
  • There are output examples of commands
  • There are screenshots with expected visualizations

@destogl destogl added documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed labels Feb 2, 2023
@destogl destogl self-assigned this Feb 2, 2023
@christophfroehlich
Copy link
Contributor

@destogl Do we need the camera links/meshes on the rrbot? or should we also clean this up ans simplify the URDF files?

@destogl
Copy link
Member Author

destogl commented Feb 23, 2023

@christophfroehlich everything is merged. Let's clean this up a bit and then we merge it into master.

@destogl Do we need the camera links/meshes on the rrbot? or should we also clean this up ans simplify the URDF files?

No we don't need this. Would be great to remove it.

destogl and others added 11 commits March 8, 2023 14:00
* Create a demo that uses transmissions (#226)

* add copy for transmission demo and set in the urdf

* Renamed to rrbot_transmissions

* Changed reduction value of transmission2

* Added the rrbot_transmissions_system_position_only

Exact copy of rrbot_system_position_only for now. Using it
from the transmission demo

* Moved logger into its own variable

* Uncrustify

* Renamed hw interface to joint. Added actuator interfaces

* Formatting. Remove comments

* Using transmissions for joint and actuator interfaces

Added a helper object to manage the joint to transmissions to actuator
data handling

* Propagating data between all the interfaces

* Cleanup. Added some consts

* Simulate motor motion

* Added logging to periodically show interface data

* Refactored the parameters a bit. Removed unused ones

* Added try/catches where needed

Checking joint consistency in the description not required. Already done
by the parser

* Added transmission demo to README

* Use clang-format instead

* rename transmissions ros2_control macro

* Remove unnecessary actuator parameters

* Typo

* Removed gazebo, mock hardware and fake sensors from the transmission
demo

* Missing 'explicit' from constructor

* Further cleanup of gazebo, mock hardware and fake sensors from the transmission demo

* branding fix

* Remove pointer definitions

Co-authored-by: Bence Magyar <[email protected]>

* Fixed comment about reserve

Co-authored-by: Bence Magyar <[email protected]>

* Renamed member variable and added description

---------

Co-authored-by: Noel Jimenez <[email protected]>
Co-authored-by: Bence Magyar <[email protected]>
(cherry picked from commit 1007b1d)

* Fix preformatted block

* fix header guard

---------

Co-authored-by: Jordan Palacios <[email protected]>
Co-authored-by: Bence Magyar <[email protected]>
* First compilable version

* Fix wrong paths

* Fix rst

* Update copyright

Co-authored-by: Dr. Denis <[email protected]>

* Restructure cmakelists

* Add maintainer

* Cleanup old paths

* Update Dr. Denis' Email and add him as author.

* Update maintainer

* Use test_nodes from ros2_controllers

* Fix trailing whitespaces

* Fix rviz display

* Add image to readme

---------

Co-authored-by: Dr. Denis <[email protected]>
@destogl destogl force-pushed the new-structure-of-master branch 2 times, most recently from 5d2bd1c to 3ae05a4 Compare March 8, 2023 13:05
@destogl destogl force-pushed the new-structure-of-master branch from 3ae05a4 to dddf1a2 Compare March 8, 2023 13:06
@destogl destogl force-pushed the new-structure-of-master branch from dddf1a2 to 6b354fd Compare March 8, 2023 13:09
@destogl destogl merged commit 6b354fd into master Mar 8, 2023
@destogl destogl deleted the new-structure-of-master branch March 8, 2023 13:11
@destogl destogl mentioned this pull request Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
2 participants