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

Readme troubleshooting references DX100 controller #25

Closed
ted-miller opened this issue May 31, 2023 · 2 comments · Fixed by #26
Closed

Readme troubleshooting references DX100 controller #25

ted-miller opened this issue May 31, 2023 · 2 comments · Fixed by #26
Labels
documentation Improvements or additions to documentation rv:0.1.0 Reported in MotoROS2 0.1.0 todo Not an issue, just a TODO
Milestone
0.1.1

Comments

@ted-miller
Copy link
Collaborator

ted-miller commented May 31, 2023 •
edited by gavanderhoorn
Loading

Just noticed this mentions DX100. Bad copy/paste from MotoROS1.

https://github.com/Yaskawa-Global/motoros2/blob/5c63671df6c6905a5e5f0aa463bec18cef5087c8/doc/troubleshooting.md#alarm-8003100---111

I'll submit a fix later this afternoon.
@ted-miller ted-miller added documentation Improvements or additions to documentation todo Not an issue, just a TODO labels May 31, 2023
@gavanderhoorn gavanderhoorn added the rv:0.1.0 Reported in MotoROS2 0.1.0 label May 31, 2023
@gavanderhoorn gavanderhoorn added this to the 0.1.1 milestone May 31, 2023
@gavanderhoorn
Copy link
Collaborator

gavanderhoorn commented May 31, 2023 •
edited
Loading

Technically it's not necessarily completely wrong, right? Just that we don't support DX100.

DX100 is also mentioned in the text for 8003[10] and 4430[6].

@ted-miller ted-miller changed the title Readme troubleshooting is wrong Readme troubleshooting references DX100 controller May 31, 2023
@ted-miller
Copy link
Collaborator Author

I think the 4430[6] one is valid to keep in

This alarm occurs if you run the DX100 version of the INIT_ROS Inform job on a non-DX100 controller.
Please delete the INIT_ROS job from your pendant and replace it with the version for all other controllers.

@ted-miller ted-miller mentioned this issue May 31, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation rv:0.1.0 Reported in MotoROS2 0.1.0 todo Not an issue, just a TODO
Projects
None yet
Milestone
0.1.1
Development

Successfully merging a pull request may close this issue.

Remove DX100 troubleshooting steps from README ted-miller/motoros2
2 participants
@gavanderhoorn @ted-miller