Closed njzjz closed 1 week ago
deepmd/utils/argcheck.py
75-75: No explicit `stacklevel` keyword argument found (B028) --- 1171-1171: Local variable `link_lf` is assigned to but never used (F841) Remove assignment to unused variable `link_lf` --- 1172-1172: Local variable `link_se_e2_a` is assigned to but never used (F841) Remove assignment to unused variable `link_se_e2_a` --- 1173-1173: Local variable `link_se_e2_r` is assigned to but never used (F841) Remove assignment to unused variable `link_se_e2_r` --- 1174-1174: Local variable `link_se_e3` is assigned to but never used (F841) Remove assignment to unused variable `link_se_e3` --- 1175-1175: Local variable `link_se_a_tpe` is assigned to but never used (F841) Remove assignment to unused variable `link_se_a_tpe` --- 1176-1176: Local variable `link_hybrid` is assigned to but never used (F841) Remove assignment to unused variable `link_hybrid` --- 1177-1177: Local variable `link_se_atten` is assigned to but never used (F841) Remove assignment to unused variable `link_se_atten` --- 1178-1178: Local variable `link_se_atten_v2` is assigned to but never used (F841) Remove assignment to unused variable `link_se_atten_v2`
Attention: Patch coverage is 70.00000%
with 3 lines
in your changes missing coverage. Please review.
Project coverage is 82.72%. Comparing base (
a09738c
) to head (b60f534
). Report is 1 commits behind head on devel.
Files | Patch % | Lines |
---|---|---|
deepmd/pt/train/training.py | 62.50% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
im confused by the relationship between the options
enable_profiler
andprofiling
. could you please explain more in the doc?
I updated the documentation. While the behavior of the two options never changed, I am wondering whether we should give a more clear name.
Summary by CodeRabbit
New Features
Documentation