Author Topic: DigiXAddons on GitHub?  (Read 5021 times)

PeterF

  • Hero Member
  • *****
  • Posts: 881
DigiXAddons on GitHub?
« on: September 18, 2013, 03:38:19 am »
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
« Last Edit: September 18, 2013, 04:11:06 am by pfeerick »

Docedison

  • Newbie
  • *
  • Posts: 4
Re: DigiXAddons on GitHub?
« Reply #1 on: September 18, 2013, 12:05:19 pm »
Item #4 Arduino IDE Ver 1.53 is replaced by Ver 1.54r2 and it does fix a lot of issues.


Doc

PeterF

  • Hero Member
  • *****
  • Posts: 881
Re: DigiXAddons on GitHub?
« Reply #2 on: September 19, 2013, 03:26:39 am »
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
« Last Edit: September 19, 2013, 03:58:30 am by pfeerick »

digistump

  • Administrator
  • Hero Member
  • *****
  • Posts: 1465
Re: DigiXAddons on GitHub?
« Reply #3 on: September 27, 2013, 12:06:31 am »
All of this fixed in newest download - http://digistu.mp/digix