search

toinsson / pyrealsense

Cross-platform ctypes/Cython wrapper to the librealsense library (v1.x)
http://pyrealsense.readthedocs.io
Apache License 2.0
121 stars 46 forks source link

Fixing examples by initializing with streams #63

Open OrganicIrradiation opened 6 years ago

OrganicIrradiation commented 6 years ago

Neither the show_cv2 nor the show_matplotlib examples were working for me with an Intel RealSense R200 without initializing the streams.

toinsson commented 6 years ago

hello, thanks for the contribution.

I am a bit surprised about not having the streams initialised. At: https://github.com/toinsson/pyrealsense/blob/1b327915ee8e7a28d17c0446a720a59356fc8a40/pyrealsense/core.py#L165 and later: https://github.com/toinsson/pyrealsense/blob/1b327915ee8e7a28d17c0446a720a59356fc8a40/pyrealsense/core.py#L196 The default streams are enabled. Do you know what is going on with R200?

Related to issue #54, thanks for the new streams. Might be valuable to put them in stream.py.

OrganicIrradiation commented 6 years ago

Hmm, I'll check it out more in depth this upcoming week and get back to you about the initialization of the default streams. Let's hold off on the PR until then.