Closed njzjz closed 2 months ago
source/tests/consistent/descriptor/test_dpa1.py (4)
`111-118`: The changes to the `is_meaningless_zero_attention_layer_tests` method look good. The logic correctly checks if the `attn_layer` is zero and any of the other conditions are true. --- `141-146`: The changes to the `skip_pt` property look good. The method call to `is_meaningless_zero_attention_layer_tests` with the specific parameters is correctly implemented. --- `169-174`: The changes to the `skip_dp` property look good. The method call to `is_meaningless_zero_attention_layer_tests` with the specific parameters is correctly implemented. --- `199-213`: The changes to the `skip_tf` property look good. The method call to `is_meaningless_zero_attention_layer_tests` with the specific parameters is correctly implemented.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.49%. Comparing base (
2bf0769
) to head (aeaeff2
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The tests make no sense in this case.
Summary by CodeRabbit
temperature
parameter to the attention layer tests.