Skip to content

Xfail tests for manipulation dims under HIP/ROCm#4662

Merged
mergify[bot] merged 2 commits intocupy:masterfrom
takagi:test-hip-manipulation-dims
Mar 4, 2021
Merged

Xfail tests for manipulation dims under HIP/ROCm#4662
mergify[bot] merged 2 commits intocupy:masterfrom
takagi:test-hip-manipulation-dims

Conversation

@takagi
Copy link
Contributor

@takagi takagi commented Feb 13, 2021

Rel #4132, #4484.

@kmaehashi kmaehashi added prio:high hip Topic: AMD ROCm / HIP labels Feb 15, 2021
]

def _check_broadcast_arrays_hip_skip_condition(self):
return self.shapes in self._broadcast_arrays_hip_skip_condition
Copy link
Member

@emcastillo emcastillo Feb 18, 2021

Choose a reason for hiding this comment

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

We can put all together in a function? 😄

def _hip_skip_invalid_broadcast(self):
    invalid_shapes = [
        [(1,), (1,)],
        [(2,), (2,)],
    ]
    if runtime.is_hip and self.shapes in invalid_shapes:
        pytest.xfail('HIP/ROCm may have a bug')

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed as such!

@emcastillo emcastillo added the st:awaiting-author Awaiting response from author label Feb 18, 2021
@takagi takagi removed the st:awaiting-author Awaiting response from author label Feb 26, 2021
@takagi
Copy link
Contributor Author

takagi commented Mar 3, 2021

pfnCI, test this please.

@chainer-ci
Copy link
Member

Jenkins CI test (for commit 2d5679e, target branch master) succeeded!

@emcastillo emcastillo added the st:test-and-merge (deprecated) Ready to merge after test pass. label Mar 4, 2021
@mergify mergify bot merged commit 91155e1 into cupy:master Mar 4, 2021
@emcastillo emcastillo added this to the v9.0.0rc1 milestone Mar 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hip Topic: AMD ROCm / HIP prio:high st:test-and-merge (deprecated) Ready to merge after test pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants