DigiXAddons on GitHub?

[PeterF]

Hi,

Have just received my beta DigiX board via post (and have been busily soldering up the bonus board for the last 20-30 mins), and when starting off with the DigiXAddons package noticed a few bugs. Am more then happy to submit pull requests to squash bugs and hopefully improve the product. Will it be hosted on github so changes can be easily made?

Namely,

1) the "DigiX Driver\Install Driver - 32-bit.bat" calls "dpinst-amd64.exe" instead of "dpinst-x86.exe"
2) the LCD demo has comments on the wrong line - backlight() doesn't print to the screen!!!
3) strange glitch in the Arduino 1.5.3 environment - the examples folder in the MPU6050 library needs to be all lower case - the initlal capital 'E' adds another tier to the menu
4) the MPU6050_DMP6 example doesn't seem to work out of the box, but MPU6050_raw does, so needs to be documented on the wiki and/or fixed
5) the 'hardware\arduino' folder should probably have been called 'hardware\digix' to avoid confusion with the arduino hardware core
btw, yet another great product guys. Keep up the hard work!

Cheers,
Peter

[Docedison]

Item #4 Arduino IDE Ver 1.53 is replaced by Ver 1.54r2 and it does fix a lot of issues.


Doc

[PeterF]

Item #4 Arduino IDE Ver 1.53 is replaced by Ver 1.54r2 and it does fix a lot of issues.

Doc

Doc, thanks for the heads up, hadn't noticed the newer version. Just installed it, haven't had time to test to see if it fixes anything else but didn't affect the MPU6050_DMP6 gremlin. Did some comparing and realised that the (non-functional) DMP6 version calls 'Wire.begin();' whereas the functional '_raw' version calls 'Wire1.begin()'. Also, due to changes to how interrupts work, the attachInterrupt call needs to change. Now connects to MPU6050 and starts outputting data, but is gobbledygook (something to do with 'InvenSense teapot demo'). For some more intelligible output, comment out the OUTPUT_TEAPOT define and uncomment OUTPUT_READABLE_YAWPITCHROLL

Cheers,
Pete

Change #158 of MPU6050_DMP6.ino from

Code: [Select]

Wire.begin();to

Code: [Select]

Wire1.begin();
Change #198 of MPU6050_DMP6.ino from

Code: [Select]

attachInterrupt(0, dmpDataReady, RISING);to

Code: [Select]

attachInterrupt(2, dmpDataReady, RISING);
Comment #112 of MPU6050_DMP6.ino

Code: [Select]

//#define OUTPUT_TEAPOT
Uncomment #95  of MPU6050_DMP6.ino

Code: [Select]

#define OUTPUT_READABLE_YAWPITCHROLL

[digistump]

All of this fixed in newest download - http://digistu.mp/digix