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

xesc2tcp script to socat xesc serial #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jkaflik
Copy link

@jkaflik jkaflik commented Aug 9, 2023

simple script. useful when playing with vesc tool

@ClemensElflein
Copy link
Owner

Thank you for the PR!
I do like the script. it's basically this script just in good: https://github.com/ClemensElflein/open_mower_ros/blob/main/utils/scripts/redirect_serial.sh

I think we should replace the script linked above with this instead of putting it into the OS. This makes it easier to update. Also forks (who might be connecting the ESCs to other ports) can overwrite it. Also we could theoretically read the mapping from ESC to /dev from the environment, but since it's hardware connected on the OM mainboard, I think it's fine like this.

What do you think?

@jkaflik
Copy link
Author

jkaflik commented Aug 13, 2023

@ClemensElflein I haven't noticed file you posted 🙈 putting it to ROS workspace is fine

docgalaxyblock added a commit to ClemensElflein/OMIAUH that referenced this pull request Sep 14, 2023
- pico fw update
- start ros bash
- xesc2tcp by jkaflik (ClemensElflein/OpenMowerOS#5)
docgalaxyblock added a commit to ClemensElflein/OMIAUH that referenced this pull request Sep 14, 2023
- pico fw update
- start ros bash
- xesc2tcp by jkaflik (ClemensElflein/OpenMowerOS#5)
docgalaxyblock added a commit to ClemensElflein/OMIAUH that referenced this pull request Jan 31, 2024
- pico fw update
- start ros bash
- xesc2tcp by jkaflik (ClemensElflein/OpenMowerOS#5)
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.

2 participants