Skip to content

Conversation

@1e-to
Copy link
Contributor

@1e-to 1e-to commented Mar 11, 2021

info::device::image2d_max_width (size_t)
info::device::image2d_max_height (size_t)
info::device::image3d_max_width (size_t)
info::device::image3d_max_height (size_t)
info::device::image3d_max_depth (size_t)

@1e-to 1e-to linked an issue Mar 11, 2021 that may be closed by this pull request
5 tasks
@1e-to 1e-to requested review from diptorupd and oleksandr-pavlyk and removed request for diptorupd March 11, 2021 11:37
Copy link
Contributor

@diptorupd diptorupd left a comment

Choose a reason for hiding this comment

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

Rebase is needed with master and the C API test cases need to be slightly changed based on what I did in #337. Otherwise good to go.

DPCTLSyclDeviceRef DRef = nullptr;
size_t image_2d_max_width = 0;
EXPECT_NO_FATAL_FAILURE(DRef = DPCTLDevice_CreateFromSelector(DSRef));
if (!DRef)
Copy link
Contributor

Choose a reason for hiding this comment

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

I changed the test cases in master. I moved the DPCTLDevice_CreateFromSelector into the fixture constructor and the GTEST_SKIP to the SetUp(). Reduces the redundant code in each test case. Please update these test cases in the same way.

return hasAspect;
}

#define declmethod(FUNC, NAME) \
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@oleksandr-pavlyk oleksandr-pavlyk left a comment

Choose a reason for hiding this comment

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

Need to be squashed.

@diptorupd diptorupd merged commit ffd1042 into IntelPython:master Apr 7, 2021
@1e-to 1e-to deleted the descriptor_image2d3d branch November 3, 2021 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add device descriptors: imageXd_max_X

3 participants