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
G-API: Expose explicit preprocessing for IE Backend #23786
G-API: Expose explicit preprocessing for IE Backend #23786
Conversation
8fc145a
to
4392782
Compare
will be applied to all input layers. | ||
@return reference to this parameter structure. | ||
*/ | ||
Params<Net>& cfgInputLayout(std::string layout) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't be exposed to python because IE
backend is going to be deprecated soon...
const cv::gapi::ie::TraitAs hint, | ||
const IE::Layout layout) { | ||
const auto &sz = mat.size; | ||
if (sz.dims() == 2 && hint == cv::gapi::ie::TraitAs::IMAGE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sz.dims() == 2
is not even needed there (Image
by design 2d
) but it also used from our tests
cv::util::optional<cv::Rect>{}); | ||
std::cout << std::endl; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@@ -1301,6 +1452,13 @@ static void configureOutputPrecision(const IE::OutputsDataMap &outputs | |||
); | |||
} | |||
|
|||
static void configureOutputLayout(const IE::OutputsDataMap &outputs_info, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe it's not even worth a separate function...
Co-authored-by: Dmitry Matveev <dmm1989@gmail.com>
@asmorkalov is there any chance to merge it? |
requires a little fix, moved to WIP so far... |
Done |
Yes, please merge |
…essing-to-ie-backend G-API: Expose explicit preprocessing for IE Backend opencv#23786 ### Pull Request Readiness Checklist See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request - [ ] I agree to contribute to the project under Apache 2 License. - [ ] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV - [ ] The PR is proposed to the proper branch - [ ] There is a reference to the original bug report and related work - [ ] There is accuracy test, performance test and test data in opencv_extra repository, if applicable Patch to opencv_extra has the same branch name. - [ ] The feature is well documented and sample code can be built with the project CMake
Pull Request Readiness Checklist
See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request
Patch to opencv_extra has the same branch name.