search

VisionLabs / torch-opencv

OpenCV bindings for Torch.
MIT License
209 stars 48 forks source link

OpenCV native bindings #110

Open edgarriba opened 8 years ago

edgarriba commented 8 years ago

I know that there's a lot of advanced work here but currently OpenCV provides macros in order to easily generate bindings for different languages. These macros makes easy the maintenance each time a signature is added/modified, so my question is if at some point was considered to generate native bindings for Torch in OpenCV?

shrubb commented 8 years ago

Thanks for raising this. Yes, I considered this approach in the first place. After some manual work, I decided to drop this idea at that time, because

1) For this, I would need to read through and understand a 1000-line Python script, 2) There appeared to be lots of lots of corner cases, 3) This would require really thorough testing.

But now, as I have a more or less clear picture of how the whole thing works, I think it's a great idea, and yes, it's inevitable that we'll have to go this way at some point. Although, I'm afraid that right now I'm not quite in condition to start such work (must complete my grad first...), maybe later.

edgarriba commented 8 years ago

agree with you. I'll start to study which is the proper way to do it. Maybe someone from OpenCV could help with this. @vpisarev @alalek