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

Bebopfrontcamera #1313

Merged
merged 1 commit into from
Sep 3, 2015
Merged

Bebopfrontcamera #1313

merged 1 commit into from
Sep 3, 2015

Conversation

dewagter
Copy link
Member

black and white streaming

@flixr
Copy link
Member

flixr commented Aug 26, 2015

Nice, will be great to finally have access to the front camera!
But the bebop_front_camera.xml module (or rather the use of viewvideo) will conflict with the video_rtp_stream.xml.... so you can't have both at the same time.

@dewagter
Copy link
Member Author

The camera interface will still be updated hopefully with ISP processing of the image instead of user space processing, but this already works. Will update naming to make it hopefully compatible (code-wise)

@dewagter
Copy link
Member Author

Now you can have both at the same time. Takes quite some processing and bandwidth though. But it compiles and runs nicely on my desk.

@gautierhattenberger
Copy link
Member

not really the time to test it, but it looks good

#define DEMOSAIC_H

/**
* @brief Bilinear demosaicing
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this from? Did you guys write that from scratch?
Just want to make sure you didn't forget some license/copyright header here....

dewagter added a commit that referenced this pull request Sep 3, 2015
Bebop front camera support in color
@dewagter dewagter merged commit f4e40b5 into master Sep 3, 2015
@dewagter dewagter deleted the bebopfrontcamera branch September 3, 2015 15:44
@flixr flixr added this to the v5.8 milestone Nov 22, 2015
@flixr flixr added the Module label Nov 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants