LCD support, misc. other fixes

ecto / duino

:bulb: Arduino framework for node.js

958 stars 214 forks source link

LCD support, misc. other fixes #31

Closed natevw closed 11 years ago

natevw commented 11 years ago

Would love to see these changes integrated into trunk. On my branch I have:

updated the documentation for code as I found it (message types, Ping plugin)
fixed a couple off-by-one assignments I noticed in du.ino
added LCD support by porting LiquidCrystal to a JS plugin (fixes #15, no protocol changes needed)
documented LCD support, updated package.js

Big thanks to &yet for sponsoring this open source work!

ecto commented 11 years ago

Wow, this is awesome! I'm pretty impressed that it was only a small change to du.ino. I'm traveling right now, so I'll take a deeper look at this when I get back. Really appreciate your effort on this.

sockdrawermoney commented 11 years ago

@ecto is this likely to get merged in?

ecto commented 11 years ago

Hey, sorry for the horrifically long wait. I was traveling again, moving, working, etc. and forgot about this. Merging now :)

ecto commented 11 years ago

Also hell yeah. Great work

natevw commented 11 years ago

Thanks, glad you like it! (And thanks again @andyet / @adambrault for sponsoring the project :-)