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

Segment faults on 32bit machine due to Alignment issues from Eigen #365

Open
dqyi11 opened this issue Apr 5, 2018 · 4 comments
Open

Segment faults on 32bit machine due to Alignment issues from Eigen #365

dqyi11 opened this issue Apr 5, 2018 · 4 comments
Assignees
Labels

Comments

@dqyi11
Copy link
Contributor

dqyi11 commented Apr 5, 2018

A few unit tests failed on 32bit machine due to segment faults.

The following tests FAILED:
14 - test_FrameDifferentiable (OTHER_FAULT)
21 - test_RnBoxConstraint (OTHER_FAULT)
22 - test_SE2BoxConstraint (OTHER_FAULT)
23 - test_RnConstantSampler (OTHER_FAULT)
25 - test_FiniteSampleable (OTHER_FAULT)
26 - test_CyclicSampleable (OTHER_FAULT)
33 - test_CartesianProductProjectable (OTHER_FAULT)
43 - test_BarrettHandKinematicSimulationPositionCommandExecutor (OTHER_FAULT)
52 - test_ParabolicTimer (OTHER_FAULT)
64 - test_VectorFieldPlanner (OTHER_FAULT)
Errors while running CTest

They are caused by alignment issues from Eigen.
https://eigen.tuxfamily.org/dox/group__DenseMatrixManipulation__Alignement.html

@dqyi11 dqyi11 added the bug label Apr 5, 2018
@dqyi11 dqyi11 self-assigned this Apr 5, 2018
@dqyi11
Copy link
Contributor Author

dqyi11 commented Apr 20, 2018

Resolved in PR #368

@jslee02
Copy link
Member

jslee02 commented Apr 20, 2018

Are all the tests are fixed? I remember there were some remaining issues.

@brianhou
Copy link
Contributor

If I remember correctly, there are still alignment issues but @dqyi11 believes that they're in DART rather than AIKIDO at this point. If that's the case, maybe we should try to create a minimal test case that only uses DART to help with debugging.

I think we also concluded that since it's only (apparently) a problem on 32-bit machines, we shouldn't invest too much time into it. But now that @dqyi11 is really good at sniffing out those issues, it might be worth it for the two of you to set up a short debugging session to see if you can fix it quickly.

@jslee02
Copy link
Member

jslee02 commented Apr 21, 2018

@brianhou Thanks for the recalling the story. In that case, I would like to leave this issue open until we actually figure out whether if this is AIKIDO's issue or not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants