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

Computer vision framework #1328

Merged
merged 6 commits into from
Sep 4, 2015
Merged

Computer vision framework #1328

merged 6 commits into from
Sep 4, 2015

Conversation

dewagter
Copy link
Member

@dewagter dewagter commented Sep 3, 2015

instead of having many new copies of the same, a computer vision framework is initiated:

video_thread

which can run one or more computer vision modules (e.g. rtp_stream or qrcode or opticflow)

@flixr flixr changed the title Computervisionframework Computer vision framework Sep 3, 2015
*/

/**
* @file modules/computer_vision/cv.h
Copy link
Member

Choose a reason for hiding this comment

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

plz fix file path
also please make use two spaces indent

@flixr
Copy link
Member

flixr commented Sep 3, 2015

I guess the viewvideo_nps.c could be removed (it doesn't do anything anyway) and the modules disabled for the nps target...

@dewagter
Copy link
Member Author

dewagter commented Sep 3, 2015

I would like to implement viewvideo_nps instead of removing it.

@dewagter dewagter mentioned this pull request Sep 3, 2015
@flixr
Copy link
Member

flixr commented Sep 3, 2015

even better, although I wonder a bit what the random image will be good for ;-)

@podhrmic
Copy link
Member

podhrmic commented Sep 3, 2015

Maybe you can somehow link it with flight gear and get some kind of video from the "cabin"?

@flixr
Copy link
Member

flixr commented Sep 3, 2015

Looks good to me, without having had the chance to actually test it....

@dewagter
Copy link
Member Author

dewagter commented Sep 4, 2015

@flixr obviously the test image is not very useful indeed, but allows checking of the computer vision routines in simulation. Note that the video_thread code from AP can also run in linux, showing your webcam image while simulating. Not sure how this should be done when you have 3 options for the simulation code: 1) (no vision in nps) 2) video_thread_nps (test image or flightgear/ROS/smartuav coupling), 3) webcam.

@dewagter dewagter merged commit cf7b87a into master Sep 4, 2015
dewagter added a commit that referenced this pull request Sep 4, 2015
Conflicts:
	conf/airframes/BR/bebop_indi.xml
	sw/airborne/modules/computer_vision/viewvideo.c
@dewagter dewagter deleted the computervisionframework branch September 4, 2015 09:52
@flixr flixr added the Module label Nov 22, 2015
@flixr flixr added this to the v5.8 milestone 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