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

Outline recommended or tested board/firmware combinations #1

Open
erelson opened this issue Apr 20, 2020 · 2 comments
Open

Outline recommended or tested board/firmware combinations #1

erelson opened this issue Apr 20, 2020 · 2 comments

Comments

@erelson
Copy link

erelson commented Apr 20, 2020

I've been trying out this library a bit the past two weekends. Really excited that someone else was able to fill this need! And the documentation is pretty great, too.

It seems to me, based on a couple failures so far, that the top-level README should probably note which board(s) is used and what firmware build?

In my case, I'm using a pyboard v1.1; I suspect you folks are working with the pyboard D-series (e.g. has wifi). esp32 boards

  • First thing I ran into: Realized I needed a threading-supporting build of firmware for my pyboard. Got v1.12 for pyboard1.1 with that from: http://micropython.org/download/pybv1/
  • Added conditionals for sys.platform == 'pyboard'
  • Now I get an error upon doing uros.NodeHandle(4, 115200) for example, regarding extra kwargs being given. This, I tracked down to uros' core.py, and apparently the firmware I grabbed is one of those that does not support the txbuf kwarg.

Now to continue! Will try modifying core.py on my end 👍

@sasilva1998
Copy link
Member

Thank you for your heads up, will improve that part in the following days! Sorry for late response. @erelson

@sasilva1998
Copy link
Member

sasilva1998 commented Aug 10, 2020

I am really sorry @erelson have been really busy with university, I am now going through an update for this lib. I am now going with the following:

  • Adding kwargs as you suggested.

  • For the txbuf parameter, I added this since I was having problems before with some Dynamixels libraries I coded, and found that assigning 0 to txbuf basically makes a flush to the serial, which makes the communication better I believe, gotta say I am not really sure, just got used it working like that (will figure out a way to change it)

  • Regarding the platform condition, I don't have a pyboard, so can't really test for it, however if you can, I would be happy to accept a PR from you.

Also yeah, I am using coded all this using a esp32 with micropython v1.12. Have a good one and thanks again. 😃

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

No branches or pull requests

2 participants