Ask Your Question
0

Getting started on contributing to Feature #3380

asked 2014-01-30 06:45:22 -0500

rohitg gravatar image

updated 2014-01-30 06:48:50 -0500

Hello!

I'm a student and just getting started with contributing to openCV (though have a lot of experience in using openCV). I've already forked, cloned, built and played a bit with the source code. I want to work on the task 3380, and thought it'd be better to run my ideas through other developers before starting to code.

If I understand the question correctly, we essentially want the ::zeros() kind-of initializers for the Vec. In my understanding, Vec is implemented using Mat in modules/core/include/opencv2/core/matx.hpp (line 869). So essentially, I need to add another function there for initialization.

Please let me know if I'm on the right track or not. I'm really a noob in working on open source, so please forgive me for mistakes. Also, I'd really appreciate any guidance on the proper way of collaborating with developers while working on such projects.

Thanks a lot!

edit retag flag offensive close merge delete

Comments

1

First of all, I support all possible contributions, they are really needed! Besides that, if you select a first contribution, you might start with an easy one, this doesn't seem that straightforward ... but hey, maybe you already figured it out! Good luck!

StevenPuttemans gravatar imageStevenPuttemans ( 2014-01-30 06:55:05 -0500 )edit

Thanks for the tip @StevenPuttemans! I actually wanted to do an easy one, but couldn't find an easier one from the Volunteer tasks. If you have one in mind, please let me know!

rohitg gravatar imagerohitg ( 2014-01-31 08:50:04 -0500 )edit

I do think that the documentation problems are the most easiest ones to tackle to get to know the submission system! http://code.opencv.org/projects/opencv/issues?query_id=26

StevenPuttemans gravatar imageStevenPuttemans ( 2014-01-31 09:03:32 -0500 )edit

Hi @StevenPuttemans@Nghia ! well maybe adding these functions would simply overload the API (as per discussion https://github.com/Itseez/opencv/pull/2307 )..however, about the 2nd part of the task, assigning Mat<_tp, n, 1> to Vec<_tp, n>, can it be done someway? or should I implement that operator?

rohitg gravatar imagerohitg ( 2014-02-12 08:50:43 -0500 )edit

1 answer

Sort by » oldest newest most voted
2

answered 2014-01-30 07:39:06 -0500

Nghia gravatar image

I think you're on the right track, or at least the same as my interpretation. It appears the request is for something like:

Matx33f a = Matx33f::ones();
Vec3f b = Vec3f::ones(); <--- currently does not work, needs to be implemented

error: conversion from ‘cv::Matx<float, 3, 1>’ to non-scalar type ‘cv::Vec3f {aka cv::Vec<float, 3>}’ requested
     Vec3f b = Vec3f::ones();

I hope you succeed, it looks useful!

edit flag offensive delete link more

Comments

rohitg gravatar imagerohitg ( 2014-02-09 06:53:33 -0500 )edit
Login/Signup to Answer

Question Tools

Stats

Asked: 2014-01-30 06:45:22 -0500

Seen: 134 times

Last updated: Jan 30 '14