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

Suggested updates based on my experience building the robot. #1

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

Conversation

gtielke
Copy link

@gtielke gtielke commented Jan 9, 2025

These are some suggested updates and work-arounds that I hope would be helpful to others in building the "Raspberry Pi Mecanum Robot" project as described in Issue 135 of the MagPi magazine, and further explained at https://www.penguintutor.com/projects/robot-mecanum .

-The mecanum.py code was changed to use 'p' or 'P' to exit the program. It had used 'q', which conflicts with some uses of the WSAD buttons on the keyboard (often mapping 'q' to some action). This is mentioned in the video on the above website.

-The README file was updated and revised:
This was to fix some formatting errors which mucked up the example Linux code, and to explain the other changes described here.
Also, it was to suggest using the "Legacy" (Bullseye) version of the Raspberry Pi OS. When implementing the AntiMicroX software on my Raspberry Pi Zero, I found it would not work with the mecanum.py code. It appears the conflict occurs with the Bookworm version of the RaspPi OS, preventing the mecanum.py process from accessing the joystick device in use by AntiMicroX in another process. My work-around is to install the "Legacy" Bullseye version of the OS. Ideally someone will update AntiMicroX, or the RasPi OS, to fix this issue. (I don't know if using Administrator mode may also fix this issue.)

-The Shell program was updated to run from the mecanum folder. In this way the AntiMicroX and mecanum.py programs all run from the same folder that has the configuration file. This also just seems like a cleaner installation.

Run from the mecanum folder
Must use Bullseye version of Pi OS. Added details on setting execution rights and configuring AntiMicroX.  Updated scripts to run from mecanum folder.
Exit on lower or uppercase P
@penguintutor
Copy link
Owner

Why change the key to p? If this is to address an issue then it needs to be explained in the pull request.

If the software is incompatible with the current version of the OS, then it should be updated rather than requesting to use an unsupported version. I am not aware of any reason why the code would be incompatible, if it is due to incompatibility with Wayland then it may be okay to suggest switching to X instead of Wayland which would allow it to be used with the current OS.

I will investigate further - but if you can explain the problems with the recent Raspberry Pi OS that may help.

@gtielke
Copy link
Author

gtielke commented Jan 10, 2025

My apologies, this is my first and only pull request, and I can see that I should have documented it better. Thank you for your feedback. I am adding some edits to my pull request to try to make it more clear.

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