Closed iProzd closed 3 months ago
Attention: Patch coverage is 98.70968%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 82.88%. Comparing base (
58b8944
) to head (3838281
). Report is 115 commits behind head on devel.
Files with missing lines | Patch % | Lines |
---|---|---|
deepmd/dpmodel/model/spin_model.py | 94.28% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I think we can skip the dpmodel tests on the CUDA machine.
Add universal uts for all models/atomic models:
Add
rot_invariant
outdef attribute to automately determine if the variable is rotationally invariant. Addtranslated_output_def
to pt models to get translated outdef variable.Fix bugs when adding ut:
In progressing (later merged in this PR):
May need help: script module API test in model uts are excluded because of long processing time. I've tried to only get torch.jit.script once, but it still doesn't work.
Note force/virial autodiff/rot test in dipole/polar need further deduction.
Summary by CodeRabbit
Tests
TEST_DEVICE
value.Chores
CUDA_VISIBLE_DEVICES
for GitHub Actions to ensure tests run on the correct hardware.