nebgnahz / cv-rs

Rust wrapper for OpenCV (manual at this point)
https://nebgnahz.github.io/cv-rs/cv/
MIT License
203 stars 41 forks source link

Mat::zeros does not accept enum #120

Open geoeo opened 5 years ago

geoeo commented 5 years ago

Expected:

let image_type = cv::CvType::Cv8SC3; let output = cv::Mat::zeros(rows, cols, image_type);

Actual: let image_type = 17 let output = cv::Mat::zeros(rows, cols, image_type);

geoeo commented 5 years ago

If this is classified as an issue, I can take a look at it

Pzixel commented 5 years ago

I think enums are always better than mere numbers. But there is a huge refactoring of all the things so I think before #117 is resolved all other changes are frozen.

vadixidav commented 5 years ago

I can take a look at this after the refactor is done. I am spending a lot of time on it today to hopefully get through that.

vadixidav commented 5 years ago

9 days later it looks like its ready to merge. Once it is good to go I will look into this.