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

[Mocap][Multicast] solve the mocap disconnection problem becuase of IGMP protocol #641

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

tongtybj
Copy link
Collaborator

@tongtybj tongtybj commented Dec 4, 2024

What is this

Force to send a IGMP report (respnse) message from client (e.g., robot, pc) to router to keep the connection for multicast.

Details

  • send _import_report.py sends the IGMP report message. However, we need to execute this script with root permission becuase of the usage of the raw socket API. Therefore the command without root permission (e.g., rosrun) requires password every time, which makes the operation not elegant.
  • install_igmp_report.sh: that's why I create this file to install the above python file in crontab under root, which allows the system to run the python file periodicaly (i.e., every two min).

Usage:

Just run following command once in your pc or robot that needs mocap data.

$ rosrun aerial_robot_base install_igmp_report.sh

TODO:

  • create a document to sumary the development.

@sugihara-16
Copy link
Contributor

@tongtybj
Awesome!
Are there some resources you referred to accomplish this work? If so, I would appreciate it if you share them.

@tongtybj
Copy link
Collaborator Author

tongtybj commented Dec 4, 2024

The introduction of multicast and IGMP(V2):

The tool to monitor packet traffic: wireshark: https://www.wireshark.org/download.html

The draft of scripts are generated by chatgpt:

  • Prompt1: please create a python script to send the IGMP membership report
  • Prompt2: please tell me a way to use crontab to run the above script every two minutes

@sugihara-16
Copy link
Contributor

Thank you so much!

@tongtybj tongtybj merged commit 88b7930 into jsk-ros-pkg:master Dec 6, 2024
6 checks passed
@Li-Jinjie
Copy link
Contributor

Li-Jinjie commented Dec 13, 2024

It's so great that the commits in this PR were squashed into a single commit in the master branch, which makes it much easier for me to cherry-pick!

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