search

nvidia-holoscan / holoscan-sensor-bridge

NVIDIA Holoscan Sensor Bridge - Bring Your Own Sensor (BYOS) over Ethernet
Apache License 2.0
6 stars 1 forks source link

ModuleNotFoundError: No module named 'hololink.native._native' #1

Closed jaydcarlson closed 2 months ago

jaydcarlson commented 2 months ago

After installing the 1.1.0-EA release via pip install ./python, I attempt to run any example, but get the error:

ModuleNotFoundError: No module named 'hololink.native._native'

Older releases work without issue

p-ogrady commented 2 months ago

Sorry you're having trouble. The error you're seeing is because the C++ part of the framework hasn't been properly set up. If you follow the instructions in the user guide for building the container, do you still see problems?

jaydcarlson commented 2 months ago

I'm trying to build it natively without using containers. On prior Holoscan Sensor Bridge releases, I've been able to pip install ./python and it would automatically run the CMake build for the native components. Is that no longer supported?

p-ogrady commented 2 months ago

Holoscan sensor bridge is tightly coupled with Holoscan SDK, which is supported via containers; so (at this time) building outside that environment isn't supported.

p-ogrady commented 2 months ago

(Closing this item; please reach out if you have any other questions!)

jaydcarlson commented 2 months ago

Just as an epilogue, I discovered this was actually my bad: while debugging another build issue I had turned off HOLOLINK_BUILD_PYTHON in CMakeLists.txt and completely forgot to turn it back on. Doh! Running pip install ./python now correctly builds the CMake-based native code and links it together into the Python module and installs it to the system.

(So while you guys might not support running this outside of a container, it definitely works, and given the annoyances of doing development inside containers, this is definitely preferred for the bulk of development by my team.)

p-ogrady commented 2 months ago

Great news; thanks so much for the update.