-
Notifications
You must be signed in to change notification settings - Fork 606
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
fix for endian mismatch issue per boostorg/python PR #218 #292
Conversation
I can confirm that this fixes the message of the RuntimeError: FATAL: module compiled as little endian, but detected different endianness at runtime
Traceback (most recent call last):
File "<string>", line 1, in <module>
File "/home/hgaiser/dev/fizyr/ros/cv_bridge/devel_isolated/cv_bridge/lib/python3.8/site-packages/cv_bridge/__init__.py", line 35, in <module>
exec(__fh.read())
File "<string>", line 6, in <module>
SystemError: initialization of cv_bridge_boost raised unreported exception So now it was actually crashing code. The proposed change fixes this issue for me. |
RuntimeError: FATAL: module compiled as little endian, but detected different endianness at runtime see also ros-perception/vision_opencv#292 and ros-melodic-arch/ros-melodic-cv-bridge#2
RuntimeError: FATAL: module compiled as little endian, but detected different endianness at runtime see also ros-perception/vision_opencv#292 and ros-melodic-arch/ros-melodic-cv-bridge#2
@@ -37,6 +37,7 @@ PyObject* pyopencv_from(const cv::Mat& m); | |||
static int do_numpy_import( ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we update the signature to void *
? This is what I'm proposing in the equivalent ros2
patch (#346).
Closing, as this has been superceded by #323 |
Greetings,
I'm on a team maintanining arch linux packages for ROS and someone just ran into this issue. Namely, upon import the user gets:
In googling around, I found this and this which were handled by PRs boostorg/python 218 and python-pni 74. Both were the same, and appear based on this explanation.
I grepped for
import_array
, added thereturn NULL
statement, rebuilt, and I no longer get the import error.