EXR reading: support Z channel if no Y channel#19893
Closed
PolarNick239 wants to merge 2142 commits intoopencv:3.4from
PolarNick239:master
Closed
EXR reading: support Z channel if no Y channel#19893PolarNick239 wants to merge 2142 commits intoopencv:3.4from PolarNick239:master
PolarNick239 wants to merge 2142 commits intoopencv:3.4from
PolarNick239:master
Conversation
Add missing `#include <thread>` in modules/gapi/test/test_precomp.hpp
G-API: ONNX. Const input * Added const input for ONNX backend * Returned initMatrixRandu, added some comments, rebase
[G-API]: Performance tests for boundingRect * Update boundingRect() tests with the changes from fitLine() PR * Add performance tests for boundingRect * Applying comment about g_type_of_t * Addressing comments * Addressing comment: replace cmp_f by CompareF in perf.tests + add the default constructor for CompareF * Fix typo
- migrate GStreamer backed - migrate FFmpeg backend (with switch on legacy API) - cv_videoio_capture_retrieve_cb_t uses Mat type instead of number of channels
- use lazy on-demand initialization
A small clerical error of an author. The correct expression should be "xy += x_*y_;".
…re_write videoio(plugins): split capture and writer APIs
* fix the perf tests of OpenCV.js so that it can run on Node.js successfully * do not modify the CMakeLists.txt Co-authored-by: lionkun <871518554@qq.com>
[G-API] Fix bug of GArray<GArray> passing through a graph * Add test to check GArray<GArray> passing through a graph (assertion failed) * G-API: Flatten GArray<T> to std::vector<T> when capturing VCtr - Also: Fix formatting in garray.hpp * Refactored test, added valuable check * Initialize size_t Co-authored-by: Dmitry Matveev <dmitry.matveev@intel.com>
[GSoC] Added TF and PyTorch segmentation conversion cases * WIP: Added conversion of segmentation models * Added tutorial mds * remove unused
[GSoC] Added TF and PyTorch detection model conversion cases * Added object detection convertion pipeline and tutorials * Added tutorial corrections
Member
|
Thank you for the contribution! This patch should go into 3.4 branch first. Please:
Note: no needs to re-open PR, apply changes "inplace". |
Contributor
|
@PolarNick239 Friendly reminder. |
Contributor
Author
|
After configuring
I will recreate the PR from forked 3.4 branch to 3.4 upstream branch, this is much easier. |
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi, some applications can produce float32 depth maps in OpenEXR format and they can use Z channel instead of Y channel (but only Y is supported in OpenCV). And this is quite natural because Z channel is a "Distance of the front of a sample from the viewer" - see OpenEXR documentation.
Currently if you will try to read such float32 .exr image with single Z channel via
imread(path, cv::IMREAD_UNCHANGED)- empty image will be returned.The proposed fix will try to find Z channel only if no Y channel is found (i.e. this fix is backwards compatible).
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.