Skip to content
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

Remove unsupported raw10/raw12 tests #2232

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xinjin01
Copy link
Contributor

Images with a CL_UNSIGNED_INT_RAW10_EXT and CL_UNSIGNED_INT_RAW12_EXT data type are unnormalised, so the normalised tests with theses images are invalid and will be skipped.

Copy link

@lakshmih lakshmih left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, minor comment.

"image test.\n");
log_info(
"-----------------------------------------------------\n\n");
return 0;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest returning TEST_SKIPPED_ITSELF here as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your comment.
Updated.

Images with a `CL_UNSIGNED_INT_RAW10_EXT` and `CL_UNSIGNED_INT_RAW12_EXT`
data type are unnormalised, so the normalised tests with theses images
are invalid and will be skipped.

Signed-off-by: Gorazd Sumkovski <[email protected]>
Signed-off-by: Xin Jin <[email protected]>
@xinjin01 xinjin01 force-pushed the raw10_12_test_rework branch from 37efc3c to 5df0c0e Compare January 28, 2025 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants