Skip to content

rename descrpt_list to list to keep compatibility#1214

Merged
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
njzjz:njzjz-patch-2
Oct 14, 2021
Merged

rename descrpt_list to list to keep compatibility#1214
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
njzjz:njzjz-patch-2

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Oct 14, 2021

fix #1211

@njzjz njzjz linked an issue Oct 14, 2021 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Oct 14, 2021

Codecov Report

Merging #1214 (ac6c34e) into devel (4e0f5e9) will increase coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel    #1214      +/-   ##
==========================================
+ Coverage   76.04%   76.05%   +0.01%     
==========================================
  Files          91       86       -5     
  Lines        7260     7247      -13     
==========================================
- Hits         5521     5512       -9     
+ Misses       1739     1735       -4     
Impacted Files Coverage Δ
deepmd/descriptor/hybrid.py 33.33% <0.00%> (-0.41%) ⬇️
source/lib/include/neighbor_list.h
source/api_cc/include/DataModifier.h
source/api_cc/include/DeepTensor.h
source/api_cc/include/DeepPot.h
source/lib/include/SimulationRegion.h

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e0f5e9...ac6c34e. Read the comment docs.

@wanghan-iapcm wanghan-iapcm merged commit 7045ba5 into deepmodeling:devel Oct 14, 2021
njzjz added a commit to njzjz/deepmd-kit that referenced this pull request Sep 21, 2023
…ling#1214)

deepmodeling#1192

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Hybrid descriptors not working

3 participants