Skip to content

do not sort atoms in dp test#2794

Merged
wanghan-iapcm merged 4 commits intodeepmodeling:develfrom
njzjz:dp-test-sort
Sep 8, 2023
Merged

do not sort atoms in dp test#2794
wanghan-iapcm merged 4 commits intodeepmodeling:develfrom
njzjz:dp-test-sort

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Sep 6, 2023

Fix #2721.

merge devel to dp-test-sort
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz requested a review from wanghan-iapcm September 6, 2023 19:58
@njzjz njzjz linked an issue Sep 6, 2023 that may be closed by this pull request
@github-actions github-actions bot added the Python label Sep 6, 2023
@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (9712c13) 79.37% compared to head (50b7f11) 79.37%.

Additional details and impacted files
@@           Coverage Diff           @@
##            devel    #2794   +/-   ##
=======================================
  Coverage   79.37%   79.37%           
=======================================
  Files         244      244           
  Lines       25820    25823    +3     
  Branches     1519     1519           
=======================================
+ Hits        20494    20497    +3     
  Misses       4933     4933           
  Partials      393      393           
Files Changed Coverage Δ
deepmd/utils/data_system.py 84.00% <ø> (ø)
deepmd/entrypoints/test.py 55.52% <100.00%> (ø)
deepmd/utils/data.py 93.81% <100.00%> (+0.06%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@wanghan-iapcm wanghan-iapcm merged commit 956adcb into deepmodeling:devel Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] do not sort atoms in outputs of dp test

2 participants