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.

Sign up for GitHub

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

Jump to bottom

[rosserial server] implement rosservice server #615

Open
wants to merge 16 commits into
base: noetic-devel
Choose a base branch
from
Open

[rosserial server] implement rosservice server #615

wants to merge 16 commits into from

Conversation

tongtybj
Copy link
Contributor

@tongtybj tongtybj commented Mar 8, 2023

What is this

enable to receive service server from arduino by usign topic_tools::ShaeShifter

tongtybj and others added 16 commits June 4, 2022 00:25
@tongtybj
Add ServiceServer in rosserial server using topic_tools::ShapeShifter.
d636dbb
@tongtybj
User asyncspinner instead of ros timer
3e544cb
@tongtybj
Add gtest code for service server in rosserial client
e27c731
@tongtybj
Resolve the double free or corruption problem in the rosservice callb…
3dd5ff3 
…ack function.
@tongtybj
Refactor the wating timeout for service server
8215b1a
@tongtybj
Change the level of verbose for sync lost
20461b0
@tongtybj
Add arg for serial.launch
d02e265
@tongtybj
Switch the service reponse and reuqest md5sum check
dd5f2e7
@tongtybj
Separate subscriber callbacks and service server callbacks into two d…
c1ca58f 
…ifferent AsyncSpinner to run them in different threads.
@tongtybj
Send ID_TX_STOP on shutdown.
90f4543
@tongtybj
Compatible usage of boost asio dispath for different boost version
88cdb93
@mjseppa
Replace busy wait loop with condition variables
169e031
@mjseppa
Match sync timeout with rosserial_python
5d9b20e
@mjseppa
Fix subscriber typo
2a8ddee
@tongtybj
[rosserial_test] remove unecessary dependency in CMake
01b93de
@tongtybj
Merge pull request #5 from mjseppa/rosservice_server
7cd68bd 
Rosservice server condition variable wait and timing fix for rosserial_python
@tongtybj
Copy link
Contributor Author

tongtybj commented Mar 8, 2023

We use rosserial_server but not rosserial_python in the case of really fast communication between the host PC and the board. But the interval of https://github.com/ros-drivers/rosserial/blob/noetic-devel/rosserial_server/include/rosserial_server/session.h#L78 is too slow, and even 1 ms is not fast enough for C++ user.
Therefore, I revert asyncspinner for ros topics/services instead of boost::asio::deadline_timer to enable instant processing for callback queue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tongtybj @mjseppa