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

Update version of piksi_ins for odometry #1297

Open
wants to merge 2 commits into
base: benaltieri/advanced_navigation_wheel_odo
Choose a base branch
from

Conversation

denniszollo
Copy link
Contributor

This PR should pull in this version of piksi_ins which includes everything we know about.

https://github.com/swift-nav/pose_daemon_wrapper/pull/49

I expect it to work and not to crash.

Ideally, when testing approves, we should finalize this branch of pose_daemon_wrapper and just merge it to master, and then undo my hardcoded change in the .mk file.

@swiftnav-svc-jenkins
Copy link
Contributor

HITL smoke tests: v2.3.0-develop-2019080116-12-g881f37ef2

These test runs are kicked off whenever you push a new commit to this PR. All passfail metrics in these runs must pass for the hitl/pass-fail status to be marked successful.

job status

gnss-analysis results

At least one run must complete for these links to have data.

passfail

detailed

@swiftnav-svc-jenkins
Copy link
Contributor

v2.3.0-develop-2019080116-12-g881f37ef2

@benjaminaltieri
Copy link
Contributor

Still seems to be an issue on startup. This is the end of the stdout log where it seems to be setting up the settings:

Registering settings.
Odometry noise 1: 0.28
Odometry noise 2: 0.28
Odometry noise 3: 0.28
Odometry noise 4: 0.28
Device to body frame roll degrees: 0
IMU to vehicle:
 0  1  0
-1  0  0
 0  0  1
Device to body frame pitch degrees: 0
IMU to vehicle:
-1  0  0
 0  0  1
Device to body frame pitch degrees: 0
IMU to vehicle:
 0  1  0
-1  0  0
 0  0  1
Device to body frame yaw degrees: 0
IMU to vehicle:
 0  1  0
-1  0  0
 0  0  1
Device to antenna offset X: 0
Device to antenna offset Y: 0
Device to antenna offset Z: -0.12674
Stillness detection enable: 1
Sideslip constrained: 1
Soln Freq updated: 10
Output cutoff: 10
Finished setup of read/write settings.
Finished setup of read-only settings.
Failed to register all message callbacks.

@denniszollo
Copy link
Contributor Author

denniszollo commented Aug 14, 2019

i figured out what is happening and why smoothpose wouldn't start. The odometry call back in piksi_ins was called twice in the odometry PR from steve. In libsbp, we check that a node doesn't already exist before inserting into the linked list. As such, when we tried to register the callback a second time, the process exited due to bad return code from libpiksi's callback register api:
https://github.com/swift-nav/pose_daemon_wrapper/pull/50/files#diff-d71ced6a91db8310fe619c9e0d3380a3R1601
image

@swiftnav-svc-jenkins
Copy link
Contributor

HITL smoke tests: v2.3.0-develop-2019080116-13-g84456902a

These test runs are kicked off whenever you push a new commit to this PR. All passfail metrics in these runs must pass for the hitl/pass-fail status to be marked successful.

job status

gnss-analysis results

At least one run must complete for these links to have data.

passfail

detailed

@swiftnav-svc-jenkins
Copy link
Contributor

v2.3.0-develop-2019080116-13-g84456902a

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.

3 participants