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

Library separation #25

Closed
9il opened this issue Aug 16, 2016 · 6 comments
Closed

Library separation #25

9il opened this issue Aug 16, 2016 · 6 comments
Assignees

Comments

@9il
Copy link
Member

9il commented Aug 16, 2016

A user may want to use this library only for CV algorithms. Library should not force a user to install any C libraries.
Looks like DCV may be splitt to

  • algorithms / image manipulations (DCV)
  • decoding / codecs
  • visualization

Current library looks like it is oriented for the end user. Comparing with Python, it is better practice for D to have an API, which can be used to build extended functionality, e.g. to be used in other libraries and cross-platform products.

@9il 9il mentioned this issue Aug 16, 2016
@ljubobratovicrelja
Copy link
Member

I agree. I was already planning on separating visualization. So if we're gonna separate I/O also, I suppose dcv.io, and dcv.plot would be optional, included only if special version is defined? That way no C libraries would be required (except blas/lapack, but that'll be fixed with #26).

@9il
Copy link
Member Author

9il commented Aug 16, 2016

So if we're gonna separate I/O also, I suppose dcv.io, and dcv.plot would be optional, included only if special version is defined?

Special configuration. Another option is move them to separate package.

@9il
Copy link
Member Author

9il commented Aug 16, 2016

Also dub has sub-packages, but i don't know how good they work

@ljubobratovicrelja
Copy link
Member

OK, I'll investigate that. I cannot tell from this point of view what would be a best option. I'd like to work on this and #26 after we resolve #20.

@9il
Copy link
Member Author

9il commented Aug 16, 2016

imageformats does not depend on any other D or libraries, so it may be preserved, thought

@9il
Copy link
Member Author

9il commented Sep 12, 2016

Also this will allow to exclude ggplotd from dependency list. ggplotd is not available for windows users :-(. This is significant constraint for newcommers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants