search

image-rs / imageproc

Image processing operations
MIT License
736 stars 144 forks source link

Merge with `cv` crate? #600

Open ripytide opened 3 months ago

ripytide commented 3 months ago

What do you guys think about merging/porting functionality from the cv crate into this library? https://github.com/rust-cv/cv/issues/81#issuecomment-2103160073

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 commented 3 months ago

We have KAZE in the roadmap https://github.com/image-rs/imageproc/issues/340. But I'm not sure about the pinhole camera and related stuff, @theotherphil what do you think?

theotherphil commented 3 months ago

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.