Closed Chengqian-Zhang closed 1 month ago
Attention: Patch coverage is 79.48718%
with 24 lines
in your changes missing coverage. Please review.
Project coverage is 82.66%. Comparing base (
eb474d4
) to head (38fc569
).
Files | Patch % | Lines |
---|---|---|
deepmd/tf/descriptor/se_atten_v2.py | 14.28% | 24 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@iProzd thinks there is no need to redirect dpa1
to se_atten_v2
. dpa1
is equivalent to se_atten
from original definition, while se_atten_v2
is a special case of dpa1
to use strip structure. So I'm not redirecting it for now.
Do you want to update the documentation in #3828 or in this PR??
Do you want to update the documentation in #3828 or in this PR??
I think it is better to update the documentation in this PR.
could you please provide consistency UT like the se_atten descriptor?
I do not add tf "se_atten_v2"
consistency UT, because the implementation of smooth_type_embedding
is different between tf and pt/dp. But smooth_type_embedding
is always true in descriptor "se_atten_v2"
.
@iProzd could you take a look at the unit test failure?
Solve #3831 and #3139
se_atten_v2
to PyTorch and DPse_attn_v2
Summary by CodeRabbit
New Features
Documentation
Bug Fixes