PiFX icon indicating copy to clipboard operation
PiFX copied to clipboard

Not working with Node v0.10.9

Open brannondorsey opened this issue 11 years ago • 3 comments

I have implemented your code and seem to be running into some errors with the SPI module. However, I am running Node v0.10.9 on my RPi instead of v0.8.15. I don't have my setup here but can post the specific errors soon. Just wondering if you know of any problems with my Node upgrade off the top of your head that could be rendering PiFX unusable?

Also, I noticed in the README for the SPI module that you have included in this repo you had the disclaimer THIS CODE IS NOT FINISHED YET, NOTHING IS FUNCTIONING YET! and mentioned that the process was blocking. I assume that the code works to some degree because your blog post exhibits a working example. Am I correct in this assumption? Also did you/anyone else make any progress on making the SPI you used for this project asynchronous?

brannondorsey avatar Oct 11 '13 15:10 brannondorsey

I haven't visited this code in a while, but I am aware of some issues with newer versions of Node.js.

The Node-SPI module's disclaimer is incorrect in this specific repository-- it does function correctly. However, I haven't checked more recent versions of Node-SPI to see if that fixes the issues with recent versions of Node.js.

I am planning on coming back to this project in the next week or two, though I will likely be finishing up an alternate Golang implementation before revisiting the Node.js app.

If anyone else has worked with PiFX and has it running on recent versions of Node.js, feel free to submit a pull request and I can take a look.

Alternatively, you could always write bytes directly to the SPI device rather than using Node SPI. This would allow the node-spi dependency to be dropped, and the app should work fine on recent versions of Node.js. This is how an alternate version of PiFX is currently implemented in Golang.

andrewmunsell avatar Oct 11 '13 21:10 andrewmunsell

Just as a follow up-- http://github.com/andrewmunsell/lfx

andrewmunsell avatar Oct 14 '13 02:10 andrewmunsell

works for me with node -v v0.10.9

8mccm8 avatar Mar 19 '14 20:03 8mccm8