X / Y confusion in examples/mindsensors/ACCL.py

GoogleCodeExporter commented 9 years ago

The culprit is here (I don't have the sensor so I can't check):

http://code.google.com/p/nxt-python/source/browse/trunk/examples/mindsensors/ACC
L.py#13

I tried a code review for this too.

Cheers.

Original issue reported on code.google.com by christop...@gmail.com on 30 Jun 2012 at 6:16

GoogleCodeExporter commented 9 years ago

Interesting find. The mindsensors people gave me this code, so I am reluctant 
to change it as they know more about the sensor than I do. I have the sensor 
and it is labeled X/Y/Z, so I'm going to test this and find out which one is 
right. I won't be able to get back to you for at least a month, unfortunately. 
Thanks for pointing this out!

Original comment by marcus@wanners.net on 1 Jul 2012 at 12:08

Changed state: Accepted
Added labels: Component-Sensors, OpSys-All, Series-2.x, Usability

GoogleCodeExporter commented 9 years ago

It looks like the code is in error. I've fixed this in r385. Thanks!

Original comment by marcus@wanners.net on 6 Aug 2012 at 10:02

Changed state: Fixed

quang-ha / nxt-python

X / Y confusion in examples/mindsensors/ACCL.py #43