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

Merge with cv crate? #600

Open
ripytide opened this issue May 9, 2024 · 2 comments
Open

Merge with cv crate? #600

ripytide opened this issue May 9, 2024 · 2 comments

Comments

@ripytide
Copy link
Member

ripytide commented May 9, 2024

What do you guys think about merging/porting functionality from the cv crate into this library?
rust-cv/cv#81 (comment)

I'd be happy to start porting and refactoring their code if we think its a good idea. Particularly in question is how we feel about more of the 3D image stuff and camera perspective stuff.

The benefits of merging the two crates are that it centralizes effort on the the same functionality.

@cospectrum
Copy link
Contributor

cospectrum commented May 10, 2024

We have KAZE in the roadmap #340. But I'm not sure about the pinhole camera and related stuff, @theotherphil what do you think?

@theotherphil
Copy link
Contributor

This crate already contains a few computer-vision-y things, but nothing very complicated. A lot of the stuff in the CV crate is out of scope here, e.g. the vSLAM stuff. Probably easiest to review any migration proposals one by one.

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

3 participants