Skip to content
This repository has been archived by the owner on Aug 13, 2019. It is now read-only.

Add support for BNO055 9-DOF IMU #11

Open
thall4 opened this issue Feb 25, 2019 · 8 comments · May be fixed by #13
Open

Add support for BNO055 9-DOF IMU #11

thall4 opened this issue Feb 25, 2019 · 8 comments · May be fixed by #13
Labels
enhancement New feature or request

Comments

@thall4
Copy link
Collaborator

thall4 commented Feb 25, 2019

Needs to be able to detect if we run into something and stop moving, and then adjust course accordingly. Doesn't have to be anything fancy, like:

if (x_accel < 0 && motor_direction == forward)
reverse();
delay(500);
leftturn();
delay(200);
forward();
//normal operation follows
@iansan5653
Copy link
Member

The new IMU you ordered is a 9-axis. Is the 345 onboard?

@iansan5653
Copy link
Member

iansan5653 commented Feb 26, 2019

We used an ADXL345 in the airbrakes. Relevant code:
Setup:
https://github.com/usfsoar/NSL_18-19_Airbrakes/blob/master/main.ino#L64-L69

Usage (about as simple as it gets; we just pulled y-accel):
https://github.com/usfsoar/NSL_18-19_Airbrakes/blob/master/main.ino#L192-L195

@thall4
Copy link
Collaborator Author

thall4 commented Feb 26, 2019

I filled this out before I decided to order another sensor, the new one I ordered has an embedded gyro which will make it more useful probably.

@iansan5653
Copy link
Member

Yeah the new one should support this feature as well as compass. What's the new one called?

@iansan5653 iansan5653 added the enhancement New feature or request label Feb 26, 2019
@iansan5653 iansan5653 added this to the Late March Launch milestone Feb 26, 2019
@thall4
Copy link
Collaborator Author

thall4 commented Feb 26, 2019

9-DOF Absolute Orientation IMU Fusion Breakout - BNO055

@iansan5653 iansan5653 changed the title Add support for ADXL345 Add support for BNO055 9-DOF IMU Feb 26, 2019
@iansan5653
Copy link
Member

accel < 0 is not a good check btw; they would trigger any time the rover decelerates, ie, if it's climbing a hill. Not sure what the new IMU supports but some sort of significant spike in negative acceleration would be better (unless the IMU supports something better I mean).

@thall4
Copy link
Collaborator Author

thall4 commented Feb 26, 2019

Yeah I know, that was me just spitballing a basic trigger off the top of my head

@thall4 thall4 closed this as completed Feb 26, 2019
@thall4 thall4 reopened this Feb 26, 2019
@iansan5653 iansan5653 linked a pull request Feb 26, 2019 that will close this issue
@iansan5653
Copy link
Member

Started a branch feature/imu with a draft pull request so we can add the feature over time with multiple people working on it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants