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

seems deprecated in favor of dcv; should it be deleted/renamed mir-cv-deprecated? #4

Open
timotheecour opened this issue Feb 20, 2018 · 6 comments

Comments

@timotheecour
Copy link
Contributor

No description provided.

@ljubobratovicrelja
Copy link
Member

It's not deprecated, just never taken care of. Idea was to have separate repo with lower level CV algorithms, implemented in betterC style, with optional C API headers. In fact, DCV was supposed to be relying on mir-cv as dependency, providing higher level API for those lower level algorithms. #3 was first step towards that, but I got caught up in too many stuff to explain here, and unfortunately never been able to finish it since.

@9il
Copy link
Member

9il commented Feb 22, 2018

Hi @ljubobratovicrelja. Sorry about #3 stop. Lets move it forward. We would be able to optimise kernels later.

@ljubobratovicrelja
Copy link
Member

We would be able to optimise kernels later.

I agree. I'll take a look at it, see what more can be easily done and I'll try and do it in the next few weeks.

@wilzbach
Copy link
Member

Dumb questions: why don't you simply have a mir-cv subpackage in the dcv repo? That would save you a lot of troubles with keeping things in sync and people who are interested in just the betterC stuff, can do e.g.

dependency "mir-cv:mir" version="~>0.9.0"

@ljubobratovicrelja
Copy link
Member

@9il had this idea of a separate repo. I thought it was OK, since there might be people who would only be interested in this project, and not in DCV. But anyways, because of the lack of action, I think that's the last of our problems at the moment. :) IMO we could always relocate the project to DCV in future if we feel like it.

@9il
Copy link
Member

9il commented Feb 23, 2018

Yes, probably we can have a DUB sub package.

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

No branches or pull requests

4 participants