-
Notifications
You must be signed in to change notification settings - Fork 27
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
base: benaltieri/advanced_navigation_wheel_odo
Are you sure you want to change the base?
Update version of piksi_ins for odometry #1297
Conversation
HITL smoke tests: v2.3.0-develop-2019080116-12-g881f37ef2These test runs are kicked off whenever you push a new commit to this PR. All passfail metrics in these runs must pass for the job statusgnss-analysis resultsAt least one run must complete for these links to have data. passfaildetailed |
v2.3.0-develop-2019080116-12-g881f37ef2
|
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:
|
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: |
HITL smoke tests: v2.3.0-develop-2019080116-13-g84456902aThese test runs are kicked off whenever you push a new commit to this PR. All passfail metrics in these runs must pass for the job statusgnss-analysis resultsAt least one run must complete for these links to have data. passfaildetailed |
v2.3.0-develop-2019080116-13-g84456902a
|
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.