ObjectnessBing runs three models (BGR, HSV, I) and writes to disk on every call

System information (version)

OpenCV => 4.1.0
Operating System / Platform => Ubuntu 16.04 64-bit
Compiler => gcc 5.4

Detailed description

The public function bool ObjectnessBING::computeSaliencyImpl(InputArray image, OutputArray objectnessBoundingBox) calls the private function void ObjectnessBING::getObjBndBoxesForSingleImage(Mat img, ValStructVec<float, Vec4i> &finalBoxes, int numDetPerSize) which requires a CV_8UC3 image and then proceeds to run the algorithm three times! on BGR input, HSV input and gray input, before printfing timing results (which are nowhere remotely close to 300Hz, more like 6Hz) and writing proposals to disk. Why? Presumably the user wants to choose between BGR, HSV, and gray, not printf timing to the console and not save results to disk.

The paper mentions the color spaces just once and the differences between running on one or all three are negligible.

My suggestions:

allow the user to select between BGR, HSV, and gray (or all 3, called "BING-diversified", see paper)
remove timing and console printing
remove writing to disk

Steps to reproduce

Examine the source code and run the sample.

Issue submission checklist

[x] I report the issue, it's not a question
[x] I checked the problem with documentation, FAQ, open issues, answers.opencv.org, Stack Overflow, etc and have not found solution
[x] I updated to latest OpenCV version and the issue is still there
[x] There is reproducer code and related data files: videos, images, onnx, etc

opencv / opencv_contrib