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.

Sign up for GitHub

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

Jump to bottom

enable IMU #726

Open
wants to merge 5 commits into
base: UI-test-tools-Production-gen2
Choose a base branch
from
Open

enable IMU #726

wants to merge 5 commits into from

Conversation

daxoft
Copy link
Contributor

@daxoft daxoft commented Jun 29, 2022

No description provided.

@daxoft daxoft force-pushed the daxoft/UI-test-tools-Production-gen2 branch 2 times, most recently from 475992c to 4cf1969 Compare June 29, 2022 23:53
@daxoft daxoft force-pushed the daxoft/UI-test-tools-Production-gen2 branch from 4cf1969 to 8c9c7d1 Compare June 30, 2022 00:08
themarpe
themarpe approved these changes Jul 13, 2022
View reviewed changes
Copy link
Collaborator

@themarpe themarpe left a comment

Choose a reason for hiding this comment

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

LGTM if tested

For IMU we have to expose RPC (todo in the future @szabi-luxonis )

saching13
saching13 suggested changes Jul 13, 2022
View reviewed changes
Copy link
Contributor

@saching13 saching13 left a comment

Choose a reason for hiding this comment

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

  1. Add the option to Print the IMU values after the IMU update.
  2. Read those values from the queue and add a pass if values are coming in.

@daxoft daxoft force-pushed the daxoft/UI-test-tools-Production-gen2 branch from 066cdaf to b216fb9 Compare August 23, 2022 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saching13 saching13 saching13 requested changes

@themarpe themarpe themarpe approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@daxoft @themarpe @saching13