Closed njzjz closed 4 months ago
deepmd/dpmodel/descriptor/se_t.py (1)
`354-382`: Refactor of `update_sel` method aligns with PR objectives. The method now correctly handles the `train_data` and `type_map` parameters to perform neighbor statistics and update selections. The return type has been adjusted to include both the updated local data and the minimum neighbor distance, which is a useful addition for further processing.deepmd/pt/model/descriptor/se_t.py (1)
`330-358`: Refactor of `update_sel` method aligns with PR objectives. The method now correctly handles the `train_data` and `type_map` parameters to perform neighbor statistics and update selections. The return type has been adjusted to include both the updated local data and the minimum neighbor distance, which is a useful addition for further processing.
Attention: Patch coverage is 81.68317%
with 37 lines
in your changes missing coverage. Please review.
Project coverage is 82.62%. Comparing base (
8de0aba
) to head (ecb7b66
). Report is 127 commits behind head on devel.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The UT for the dp model is necessary. or shall we provide consistency test for the behavior of update_sel across backends?
Good idea. The e2e and unit tests are not in conflict.
We haven't had fixtures for descriptors, though.
@coderabbitai resolve
Fix https://github.com/deepmodeling/deepmd-kit/issues/3525. Fix https://github.com/deepmodeling/deepmd-kit/issues/3544.
Summary by CodeRabbit
update_sel
method to accept additional parameters and return more detailed data, improving model selection and neighbor statistics.hook
method to streamline codebase.