Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

157 update board interface #170

Merged
merged 7 commits into from
Jun 21, 2024
Merged

157 update board interface #170

merged 7 commits into from
Jun 21, 2024

Conversation

iandareid
Copy link
Contributor

The interface changes have been made. They assume that the sensors always have new data when queried. This hasn't caused strange behavior, and I believe that is how it was done before changing the interface. Just something that seems odd.

Now the aircraft actually loads all parameters in sim, but it won't arm.
I have not tested if all RC inputs fail, though it is likely.
The firmware appears to be functional.
The sim works for waypoint missions.
@iandareid iandareid linked an issue Jun 21, 2024 that may be closed by this pull request
@iandareid iandareid self-assigned this Jun 21, 2024
@iandareid
Copy link
Contributor Author

@JMoore5353 @bsutherland333

I am curious as to your thoughts with the having the SENSOR_has_new_data() methods always returning true.

@iandareid
Copy link
Contributor Author

There was a bug with the return true, so I adjusted it to match the imu_new_data which uses gazebo time. This can be easily switched to generic sim time when we finally disconnect gazebo.

Copy link
Contributor

@bsutherland333 bsutherland333 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I just had one comment

@iandareid iandareid merged commit a200e9c into main Jun 21, 2024
2 checks passed
@iandareid iandareid deleted the 157-update-board-interface branch June 21, 2024 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Update simulation with changes to board interface
2 participants