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

Add pip CI #386

Merged

Conversation

nim65s
Copy link
Member

@nim65s nim65s commented Mar 20, 2023

Hi,

This PR adds some CI for MacOS & Linux with pip. It shows a test failure with mac OS:

 5/33 Test  #2: collision ........................***Failed    4.49 sec
Running 5 test cases...

unknown location:0: fatal error: in "mesh_mesh": memory access violation at address: 0x8ae0b56035: no mapping at fault address
/Users/runner/work/hpp-fcl/hpp-fcl/test/collision.cpp:596: last checkpoint: "mesh_mesh" test entry

*** 1 failure is detected in the test module "FCL_COLLISION"

jcarpent
jcarpent previously approved these changes Mar 20, 2023
@jcarpent jcarpent changed the title CI: pip Add pip CI Mar 20, 2023
@jcarpent
Copy link
Member

@nim65s I'm able to reproduce the issue.
What is the purpose of -D_GLIBCXX_USE_CXX11_ABI=0?

@jcarpent
Copy link
Member

I'm on the bug, I will fix it soon.

@nim65s
Copy link
Member Author

nim65s commented Mar 20, 2023

-D_GLIBCXX_USE_CXX11_ABI=0 is a temporary workaround because cmeel binaries are currently compiled with an older GCC, following stack-of-tasks/pinocchio#1819.

A better fix with more details is on its way, but need some work before the publication.

@jcarpent
Copy link
Member

I think the new issue we encounter if due to a very recent version of Boost.Timer and Boost.Chrono.
I did have time to investigate deeply, but I rather suggest relying on std::chrono for full compatibility.

@jcarpent jcarpent merged commit 98303de into humanoid-path-planner:devel Mar 21, 2023
20 checks passed
@nim65s nim65s deleted the topic/mesh-mesh-memory-violation branch March 23, 2023 09:35
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.

None yet

2 participants