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

If the board doesn't report a firmata version, ask for it #17

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

If the board doesn't report a firmata version, ask for it #17

wants to merge 1 commit into from

Conversation

theycallmeswift
Copy link
Collaborator

It's possible that the board doesn't report the firmata version before the iterate loop in the init function ends (mostly in high latency situations like bluetooth serial). This fixes it by asking the board for a version and iterating until it receives it or until 5 seconds has elapsed.

Only thing I wasn't sure on was wether or not to raise an exception if firmata version wasn't reported. What do you think?

Many thanks to @kymwatts to helping me discover this issue.

@theycallmeswift
Copy link
Collaborator Author

FYI, there's no test here because I'm not sure about what to do if the board doesn't report. Happy to add it once we come to a conclusion.

@theycallmeswift
Copy link
Collaborator Author

Any update on this? Really need it to go in so I can get Bluetooth serial communications working.

@theycallmeswift
Copy link
Collaborator Author

Hey @tino any updates on this? I really need this to go in and haven't heard anything from you...

@tino
Copy link
Owner

tino commented Oct 2, 2013

The solution sounds right to me, and we probably want to issue a warning if we dont get what we want. I'll probably be able to test and check this in this weekend. Sorry if I aint to quick to respond, it's a hobby project for me.

On Sat, Sep 28, 2013 at 6:11 PM, Swift notifications@github.com wrote:

Hey @tino any updates on this? I really need this to go in and haven't heard anything from you...

Reply to this email directly or view it on GitHub:
#17 (comment)

@theycallmeswift
Copy link
Collaborator Author

Cool. Keep me posted on anything you need from me.

Totally understand it's a hobby project. Mine is too :) Just a friendly ping. If you ever need help maintaining, I'm more than happy to lend a hand.

@theycallmeswift
Copy link
Collaborator Author

Ping. Any updates on this? I'd really like to get it in before Robots Conf (http://robotsconf.com/)

@tino
Copy link
Owner

tino commented Nov 27, 2013

Hey man, sorry to keep you waiting for so long. I've been pushing it from weekend to weekend (together with some other related issues on this repo), but still have a lot of other stuff to do. So please feel free to implement these changes yourself.

@tino
Copy link
Owner

tino commented Oct 29, 2016

I guess I waited that long that there are conflicts now. Anybody still interested in this fix?

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