alexismeneses/py-adxl345

Python driver for ADXL345 accelerometer

py-adxl345's Issues

[Feature request] - Tap

Hello Alexis,
i'd like to get working singletap, doubletap and freefall which adxl345 provide as HW features, so i forked your project and looked an old lib [0] trying to re-create the functions used for tapping.
Unlucky i'm a self teached programmer so i was only able to add Pitch&Roll [1] to your code, whilst it get hard to me play with hex address and bitwise operations. After many tries i feel to be close to the solution but it still doesn't work.

Can you please give me an hand and try to add Tap in your code?
Thank you :)

[0] http://think-bowl.com/raspberry-pi/i2c-python-library-3-axis-digital-accelerometer-adxl345-with-the-raspberry-pi/
[1] http://theccontinuum.com/2012/09/24/arduino-imu-pitch-roll-from-accelerometer/

Typo in base.py line 285

base.py >

at line 285:
self.set_offset(offset_x, offest_y, offset_z)

must be:
self.set_offset(offset_x, offset_y, offset_z)

Otherwise

"offest_y" is not defined

nvm, sorry

nvm, sorry

Wrong code part in spi.py

spi.py >

at line 30
self.spi.writebytes( [ address & 0x3F ] | READ_MASK | MULTIREAD_MASK )
must be
self.spi.writebytes( [ address & 0x3F | READ_MASK | MULTIREAD_MASK ])

otherwise it raises TypeError

TypeError: unsupported operand type(s) for |: 'list' and 'int'