diff --git a/test/transforms_v2_kernel_infos.py b/test/transforms_v2_kernel_infos.py index 6fea2513712..4469cf1abbb 100644 --- a/test/transforms_v2_kernel_infos.py +++ b/test/transforms_v2_kernel_infos.py @@ -79,20 +79,20 @@ def __init__( self.logs_usage = logs_usage -def _pixel_difference_closeness_kwargs(uint8_atol, *, dtype=torch.uint8, mae=False): +def pixel_difference_closeness_kwargs(uint8_atol, *, dtype=torch.uint8, mae=False): return dict(atol=uint8_atol / 255 * get_max_value(dtype), rtol=0, mae=mae) def cuda_vs_cpu_pixel_difference(atol=1): return { - (("TestKernels", "test_cuda_vs_cpu"), dtype, "cuda"): _pixel_difference_closeness_kwargs(atol, dtype=dtype) + (("TestKernels", "test_cuda_vs_cpu"), dtype, "cuda"): pixel_difference_closeness_kwargs(atol, dtype=dtype) for dtype in [torch.uint8, torch.float32] } def pil_reference_pixel_difference(atol=1, mae=False): return { - (("TestKernels", "test_against_reference"), torch.uint8, "cpu"): _pixel_difference_closeness_kwargs( + (("TestKernels", "test_against_reference"), torch.uint8, "cpu"): pixel_difference_closeness_kwargs( atol, mae=mae ) } @@ -104,7 +104,7 @@ def float32_vs_uint8_pixel_difference(atol=1, mae=False): ("TestKernels", "test_float32_vs_uint8"), torch.float32, "cpu", - ): _pixel_difference_closeness_kwargs(atol, dtype=torch.float32, mae=mae) + ): pixel_difference_closeness_kwargs(atol, dtype=torch.float32, mae=mae) } @@ -1925,9 +1925,6 @@ def sample_inputs_adjust_contrast_video(): yield ArgsKwargs(video_loader, contrast_factor=_ADJUST_CONTRAST_FACTORS[0]) -# TODO: this is just temporary to make CI green for release. We should add proper tolerances after -skip_adjust_contrast_jit = TestMark(("TestKernels", "test_scripted_vs_eager"), pytest.mark.skip(reason="Test is flaky")) - KERNEL_INFOS.extend( [ KernelInfo( @@ -1941,14 +1938,16 @@ def sample_inputs_adjust_contrast_video(): **pil_reference_pixel_difference(), **float32_vs_uint8_pixel_difference(2), **cuda_vs_cpu_pixel_difference(), + (("TestKernels", "test_against_reference"), torch.uint8, "cpu"): pixel_difference_closeness_kwargs(1), }, - test_marks=[skip_adjust_contrast_jit], ), KernelInfo( F.adjust_contrast_video, sample_inputs_fn=sample_inputs_adjust_contrast_video, - closeness_kwargs=cuda_vs_cpu_pixel_difference(), - test_marks=[skip_adjust_contrast_jit], + closeness_kwargs={ + **cuda_vs_cpu_pixel_difference(), + (("TestKernels", "test_against_reference"), torch.uint8, "cpu"): pixel_difference_closeness_kwargs(1), + }, ), ] ) @@ -2064,9 +2063,6 @@ def sample_inputs_adjust_saturation_video(): yield ArgsKwargs(video_loader, saturation_factor=_ADJUST_SATURATION_FACTORS[0]) -# TODO: this is just temporary to make CI green for release. We should add proper tolerances after -skip_adjust_saturation_cuda = TestMark(("TestKernels", "test_cuda_vs_cpu"), pytest.mark.skip(reason="Test is flaky")) - KERNEL_INFOS.extend( [ KernelInfo( @@ -2079,13 +2075,13 @@ def sample_inputs_adjust_saturation_video(): closeness_kwargs={ **pil_reference_pixel_difference(), **float32_vs_uint8_pixel_difference(2), + **cuda_vs_cpu_pixel_difference(), }, - test_marks=[skip_adjust_saturation_cuda], ), KernelInfo( F.adjust_saturation_video, sample_inputs_fn=sample_inputs_adjust_saturation_video, - test_marks=[skip_adjust_saturation_cuda], + closeness_kwargs=cuda_vs_cpu_pixel_difference(), ), ] )