Skip to content

Conversation

@caternuson
Copy link
Contributor

For #34.

  • Had to bury the math in the scale parameter. Couldn't just math on the _ScaledReadOnlyStruct class. Decided not to import math just for pi.
  • Marked gyroscope as deprecated
  • Updated bno055_simpletest.py example
  • webgl example is OK as is, doesn't seem to use gyro
  • Tested with Itsy M4 and updated bno055_simpletest.py

@ladyada
Copy link
Member

ladyada commented Aug 15, 2019

@codematerial please try this!

@TheMariday
Copy link

TheMariday commented Aug 16, 2019

@ladyada I sadly can't test this as I can't install this library or any CircuitPython libraries to the NVidia Jetson but think it's due to a problem with my system and not a problem with the library.

However I've added a comment on issue #33 as I'm still getting RPS straight out of the sensor using the old library even though it should be DPS. So I worried this might be an issue here too.

Running pip3 install adafruit-circuitpython-bno055 as referenced here gives me the following error:

Command "/usr/bin/python3 -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-ug865wg6/rpi-ws281x/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-v49o2es5-record/install-record.txt --single-version-externally-managed --compile --user --prefix=" failed with error code 1 in /tmp/pip-build-ug865wg6/rpi-ws281x/

I'm not too sure why it's trying to install a RPi specific LED driver on the Jetson but I can see how this might cause issues!

@caternuson
Copy link
Contributor Author

To test locally, you can pull the PR code down and set PYTHONPATH to point that directory. That way you don't have to rely on pip.

@makermelissa
Copy link
Collaborator

I'm setting up my Jetson Nano right now to see if I can get it installed and testable.

@makermelissa
Copy link
Collaborator

Ok, @CodeMaterial, I did not get the Nvidia Jetson error you're referring to. Could you please try running pip3 install --upgrade adafruit_blinka. There have been a couple Jetson related updates in the past 24 hours and it's possible the updates fixed the issues that you're having.

@makermelissa
Copy link
Collaborator

Another option is to install Blinka using the sudo command as that installs to a different location.

@TheMariday
Copy link

Hi @makermelissa.

As this seems to be an issue with Blinka I've created a new issue here: adafruit/Adafruit_Blinka#149.

Thanks and I hope that this pull request can get tested to make sure that the units are correct for the BNO library.

Sam.

@caternuson caternuson requested a review from a team September 9, 2019 15:05
@ladyada ladyada merged commit d3edbc3 into adafruit:master Sep 9, 2019
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants