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

Add diagnostics_ethercat_summarizer.py for packet drop confirmation #1417

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Add diagnostics_ethercat_summarizer.py for packet drop confirmation #1417

wants to merge 8 commits into from

Conversation

Kanazawanaoaki
Copy link
Contributor

@Kanazawanaoaki Kanazawanaoaki commented Dec 16, 2021 •
edited
Loading

This PR adds diagnostics_ethercat_summarizer.py to make it easier to check packet drops.

$ rosrun jsk_pr2_startup diagnostics_ethercat_summarizer.py 
EtherCAT Master Dropped Packets: 0
EtherCAT Master RX Late Packet: 0
EtherCAT Device (br_caster_r_wheel_motor)Drops: 0
EtherCAT Device (br_caster_l_wheel_motor)Drops: 0
EtherCAT Device (br_caster_rotation_motor)Drops: 0
EtherCAT Device (fl_caster_r_wheel_motor)Drops: 0
EtherCAT Device (fl_caster_l_wheel_motor)Drops: 0
EtherCAT Device (fl_caster_rotation_motor)Drops: 0
EtherCAT Device (bl_caster_r_wheel_motor)Drops: 0
EtherCAT Device (bl_caster_l_wheel_motor)Drops: 0
EtherCAT Device (bl_caster_rotation_motor)Drops: 0
EtherCAT Device (fr_caster_r_wheel_motor)Drops: 0
EtherCAT Device (fr_caster_l_wheel_motor)Drops: 0
EtherCAT Device (fr_caster_rotation_motor)Drops: 0
EtherCAT Device (torso_lift_motor)Drops: 0
EtherCAT Device (l_shoulder_pan_motor)Drops: 0
EtherCAT Device (l_shoulder_lift_motor)Drops: 0
EtherCAT Device (l_upper_arm_roll_motor)Drops: 0
EtherCAT Device (l_elbow_flex_motor)Drops: 0
EtherCAT Device (l_forearm_roll_motor)Drops: 0
EtherCAT Device (l_wrist_l_motor)Drops: 0
EtherCAT Device (l_wrist_r_motor)Drops: 0
EtherCAT Device (l_gripper_motor)Drops: 0
EtherCAT Device (head_pan_motor)Drops: 0
EtherCAT Device (r_shoulder_pan_motor)Drops: 0
EtherCAT Device (r_shoulder_lift_motor)Drops: 0
EtherCAT Device (r_upper_arm_roll_motor)Drops: 0
EtherCAT Device (r_elbow_flex_motor)Drops: 0
EtherCAT Device (r_forearm_roll_motor)Drops: 0
EtherCAT Device (r_wrist_l_motor)Drops: 0
EtherCAT Device (r_wrist_r_motor)Drops: 0
EtherCAT Device (r_gripper_motor)Drops: 0
EtherCAT Device (head_tilt_motor)Drops: 0
EtherCAT Device (laser_tilt_mount_motor)Drops: 0
--------------------------------------------

Without this PR, you have to expand and check the EtherCAT Devices of the rqt_pr2_dashboard or rqt_runtime_monitor one by one.

@knorth55
Copy link
Member

please rename from diagnos_ether_listener.py to diagnosis_ethercat_summarizer or something.

@Kanazawanaoaki Kanazawanaoaki changed the title Add diagnos_ether_listener.py for packet drop confirmation Add diagnostics_ethercat_summarizer.py for packet drop confirmation Dec 20, 2021
@k-okada k-okada requested a review from knorth55 April 18, 2022 11:46
Kanazawanaoaki and others added 4 commits April 26, 2022 20:07
@Kanazawanaoaki @knorth55
Update jsk_pr2_robot/jsk_pr2_startup/jsk_pr2_warning/diagnostics_ethe…
7e7fcbc 
…rcat_summarizer.py

Co-authored-by: Shingo Kitagawa <knorth55@users.noreply.github.com>
@Kanazawanaoaki @knorth55
Update jsk_pr2_robot/jsk_pr2_startup/jsk_pr2_warning/diagnostics_ethe…
85b6be2 
…rcat_summarizer.py

Co-authored-by: Shingo Kitagawa <knorth55@users.noreply.github.com>
@Kanazawanaoaki
Remove white space
9fa5d2f
@knorth55
Merge branch 'master' into add-diagnos-listener
2c1dc73
@knorth55 knorth55 mentioned this pull request Jul 6, 2022
Closed
50 tasks
@knorth55 knorth55 mentioned this pull request Sep 11, 2022
Open
61 tasks
@Kanazawanaoaki Kanazawanaoaki mentioned this pull request Nov 4, 2022
Open
@knorth55 knorth55 mentioned this pull request Mar 25, 2023
Open
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@knorth55 knorth55 knorth55 requested changes

Assignees
No one assigned
Labels
PR1040
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Kanazawanaoaki @knorth55