Skip to content

gpio_controller fix#29

Open
Nibanovic wants to merge 78 commits intoio_gripper_controllerfrom
io_gripper_controller-fix
Open

gpio_controller fix#29
Nibanovic wants to merge 78 commits intoio_gripper_controllerfrom
io_gripper_controller-fix

Conversation

@Nibanovic
Copy link

@Nibanovic Nibanovic commented Sep 9, 2025

gpio controller won't compile with C++17 due to this type:

std::atomic<std::shared_ptr<std::string>> target_configuration_;

std_shared does nto have a trivial copy constructor which can be handled with C++17 atomic.
In this PR, I've opted for using realtime_tools::RealtimeThreadSafeBox for access to the string.

COLCON_INGORE io_gripper_controller

Consider deleting io_gripper_controller from the repo, it is empty and this replaces it.

sachinkum0009 and others added 30 commits August 16, 2025 21:11
- minor code clean
- removed the template from license
- added one variable per line
- documented the enums
- updated the doc folder as per `joint_trajectory_controller`
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
- renamed service_buffer_ to gripper_service_buffer_
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
Co-authored-by: Dr. Denis <denis@stoglrobotics.de>
@mergify
Copy link

mergify bot commented Sep 9, 2025

@Nibanovic, all pull requests must be targeted towards the master development branch.
Once merged into master, it is possible to backport to io_gripper_controller, but it must be in master
to have these changes reflected into new distributions.

@mergify
Copy link

mergify bot commented Sep 16, 2025

This pull request is in conflict. Could you fix it @Nibanovic?

@github-actions
Copy link

This PR is stale because it has been open for 45 days with no activity. Please tag a maintainer for help on completing this PR, or close it if you think it has become obsolete.

@github-actions github-actions bot added the stale label Oct 31, 2025
@destogl destogl force-pushed the io_gripper_controller branch from bf9723f to 0364c71 Compare February 25, 2026 13:20
@github-actions github-actions bot removed the stale label Feb 26, 2026
@mergify
Copy link

mergify bot commented Feb 26, 2026

This pull request is in conflict. Could you fix it @Nibanovic?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants