-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
kdl-parser-py OSX segfault #89
Comments
added! |
@sonelu it's here: https://anaconda.org/robostack/ros-noetic-kdl-parser-py Let us know how that one goes :) |
Well, I have a seg fault...:
The short dump looks like this:
And the
I'll try also with a less complex example (I'll take one from their test) and see if that does the same. |
Do you know how sip is involved (Qt somewhere?)? |
I have to look, but it's definitely not from Qt. I think the KDL library is in cpp and probably they used sip to wrap it for python use. |
I've tried running it with the simple URDF they have in the test and it also seg faults:
the shortdump is a little different though:
|
Actually the problem seems to be from the |
Would you mind filing an issue upstream? Maybe they’ve got an idea |
Note a few things:
Could we
|
For number 1 I can confirm that the seg fault is from PyKDL:
The short-dump looks the same as the ones above (the error start with the handling of the string when the Joint object is created):
For number 2 if someone has a Linux machine at hand where they can run the same code as above it would be great. Otherwise I can try to spun a VM and try it. |
This will be fixed once conda-forge/orocos-kdl-feedstock#11 is merged in. That PR also contains a test to avoid regression. |
@sonelu can you try whether this works now? We have a new build of the orocos-kdl, I'm not sure whether the ROS packages need rebuilding. |
@Tobias-Fischer I've tried a new install in a new environment and I still get the same segmentation fault as here. But the version I got from conda when installing was 1.4.0 and I noticed that you already released 1.5.0 in the orocos repo. I assume the conda builds need to be re-run to have the updated code. This is the build that was installed by conda:
|
That works now - I created a new env with |
Could we add
ros-noetic-kdl-parser-py
in the list? I see it was initially in the melodic stack:Thanks.
The text was updated successfully, but these errors were encountered: