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 random.power
test with forbidden value
#2375
Conversation
@@ -539,7 +539,7 @@ def test_power(self, a_dtype, dtype): | |||
@cupy.testing.for_float_dtypes('dtype', no_float16=True) | |||
@cupy.testing.for_float_dtypes('a_dtype') | |||
def test_power_for_zero_a(self, a_dtype, dtype): |
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.
Why not rename this test?
def test_power_for_zero_a(self, a_dtype, dtype): | |
def test_power_for_one_a(self, a_dtype, dtype): |
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.
BTW, why is this test needed?
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.
Oh I have no idea. Maybe we can just remove this test?
c45a3dd
to
ef81bc9
Compare
random.power
test with forbidden value
Removed the test. PTAL |
LGTM. pfnCI, test this please. |
Successfully created a job for commit ef81bc9: |
Jenkins CI test (for commit ef81bc9, target branch master) failed with status FAILURE. |
pfnCI, test this please. |
Successfully created a job for commit ef81bc9: |
Jenkins CI test (for commit ef81bc9, target branch master) failed with status FAILURE. |
LGTM. |
Remove `random.power` test with forbidden value
Fixes part of #2370
a=0
has been forbidden innumpy.random.power
, but it seems it only became an error in NumPy 1.17.