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

Fixing examples by initializing with streams #63

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

OrganicIrradiation
Copy link
Contributor

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

@toinsson
Copy link
Owner

toinsson commented Jan 4, 2018

hello, thanks for the contribution.

I am a bit surprised about not having the streams initialised.
At:

if streams is None:

and later:
for s in streams:

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
Copy link
Contributor Author

OrganicIrradiation commented Jan 7, 2018

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.

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.

2 participants