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

updates README to have slightly less confusing instructions #15

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

Conversation

rockbot
Copy link

@rockbot rockbot commented Nov 7, 2013

No description provided.

@prolificcoder
Copy link

@rockbot can you share the flickr images, I am having trouble with moving the bot after hackathon

@rockbot
Copy link
Author

rockbot commented Nov 8, 2013

Absolutely! Flickr images here: http://bit.ly/1baWlTV

Holler if you have questions :-)

@prolificcoder
Copy link

Seems like everything is setup on the bot according to those flickr images, but I am unable to make it dance..

This is the log.. After first dance() I see Repl initialized...

node src/bot.js
1383940470696 Board Connecting...
1383940470882 Serial Found possible serial ports /dev/cu.Bluetooth-Incoming-Port,/dev/cu.usbmodemfa141
1383940470883 Board -> Serialport connected /dev/cu.Bluetooth-Incoming-Port
dance()
1383940474981 Repl Initialized

dance()
ReferenceError: dance is not defined

@ghost
Copy link

ghost commented Nov 8, 2013

Oh hey I know what's up with that. Need to connect to the USB port instead of Bluetooth @rockbot showed me how:

Plug Tapster into USB port on your Mac

Open terminal

type '/dev/usb' then autocomplete

the output should be something like /dev/tty.usbmodem1234

Copy that output line to the board object of bot.js such as:

board = new five.Board({
  debug: false, 
  port: '/dev/tty.usbmodem1234'
});

This will allow the app to connect to the proper USB port instead of Bluetooth or whatever else may be autodetected

@rwaldron
Copy link
Contributor

rwaldron commented Nov 8, 2013

Totally unnecessary, this bug was fixed and I'm so sorry that I forgot to push to npm.

@rwaldron
Copy link
Contributor

rwaldron commented Nov 8, 2013

Just pushed 0.7.5 to npm

sorry about that!

@ghost
Copy link

ghost commented Nov 9, 2013

Awesome thanks!

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.

3 participants