已合并
[test]skip_test_jacfwd #39004
SCh_zx创建于 6月23日
[test]skip_test_jacfwd #39004
已合并
共 1 个文件变更+6-0
| @@ -1,4 +1,9 @@ | |||
| 1 | # Owner(s): ["module: functorch"] | 1 | # Owner(s): ["module: functorch"] |
| 2 | +# Copyright (c) Facebook, Inc. and its affiliates. | ||
| 3 | +# All rights reserved. | ||
| 4 | +# | ||
| 5 | +# This source code is licensed under the BSD-style license found in the | ||
| 6 | +# LICENSE file in the root directory of this source tree. | ||
| 2 | 7 | ||
| 3 | import copy | 8 | import copy |
| 4 | import math | 9 | import math |
| @@ -2474,6 +2479,7 @@ class TestHessian(TestCase): | |||
| 2474 | y = torch.randn(3, device=device) | 2479 | y = torch.randn(3, device=device) |
| 2475 | self._test_against_reference(f, (x, y)) | 2480 | self._test_against_reference(f, (x, y)) |
| 2476 | 2481 | ||
| 2482 | + | ||
 | |||
| 2477 | def test_jacfwd_different_levels(self, device): | 2483 | def test_jacfwd_different_levels(self, device): |
| 2478 | # Test case from: | 2484 | # Test case from: |
| 2479 | # pytorch functorch issues 597 | 2485 | # pytorch functorch issues 597 |
🔵 Low Priority
新增的
@unittest.skip("skip ci err jacfwd")装饰器的 skip 原因字符串过于模糊。与该文件中其他 skip 的惯例（例如"fails in Windows; needs investigation"、"see pytorch torchdynamo issue 103613"、# pytorch issues 86798）相比，"skip ci err jacfwd" 没有说明具体的失败模式、没有引用追踪 issue 编号，也没有说明在什么条件下可以重新启用该测试。这会导致后续维护者无法判断该 skip 是否仍然必要、底层问题是否已修复。此外，测试方法名为
test_jacfwd_different_levels，但测试体实际使用的是jacrev(jacrev(loss))，而非jacfwd。skip 消息中的 "jacfwd" 指向的是测试名称而非测试体实际内容，这可能意味着 skip 被加到了错误的测试上——即原本要跳过的是一个实际使用jacfwd的测试，但由于名称误导，skip 被放到了这个使用jacrev的测试上。这会不必要地降低hessian/jacrev组合正确性的测试覆盖率。建议：将 skip 消息更新为更具信息量的描述，至少应包含：(1) 引用具体的 issue 追踪编号，(2) 简要描述失败原因或错误信息。同时请确认该 skip 应当放在此测试（测试体使用 jacrev 而非 jacfwd）上，而不是其他实际使用 jacfwd 的测试上。