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

Install module "pyserial" instead of "serial" #351

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

Conversation

damien-robotsix
Copy link

I was trying to flash a Pixhawk v6x though the docker and even with appropriate rights, the upload python script was failing to connect to the board.

Digging a bit I got a reason: module 'serial' has no attribute 'Serial'

This is due to the fact that the module 'serial' is installed in the docker and not 'pyserial'.

Is this wanted? do we need the module 'serial' or is it a typo mistake?

Anyway, without the module pyserial, the upload script won't work.

Thank you for your feedback.

@damien-robotsix
Copy link
Author

A consequence is that the module future is not installed anymore (it was a dependency of serial) so I had to put it back in the pip module list to be installed as the module 'future' is required to build the firmware.

Again, this assume that the module 'serial' itself is not needed but this I am unsure. (I never needed it in my usage of the PX4 firmware and better not have it since it might conflict with pyserial).

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.

1 participant