Skip to content

fix neighbor stat mixed_types input#3304

Merged
njzjz merged 3 commits intodeepmodeling:develfrom
njzjz:fix-neighbor-stat
Feb 20, 2024
Merged

fix neighbor stat mixed_types input#3304
njzjz merged 3 commits intodeepmodeling:develfrom
njzjz:fix-neighbor-stat

Conversation

@njzjz
Copy link
Copy Markdown
Member

@njzjz njzjz commented Feb 19, 2024

It was broken by #3289, which replaced distinguished types by mixed types but didn't change the input.

Add tests for two types.

It was broken by deepmodeling#3289, which replaced distinguished types by mixed types but didn't change the input.

Add tests for two types.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz requested a review from wanghan-iapcm February 19, 2024 22:49
@njzjz njzjz marked this pull request as ready for review February 19, 2024 22:50
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ab35468) 75.05% compared to head (530a3ce) 75.05%.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #3304      +/-   ##
==========================================
- Coverage   75.05%   75.05%   -0.01%     
==========================================
  Files         396      396              
  Lines       33895    33895              
  Branches     1604     1600       -4     
==========================================
- Hits        25441    25440       -1     
- Misses       7593     7594       +1     
  Partials      861      861              

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

Comment thread deepmd/dpmodel/utils/neighbor_stat.py Outdated
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz requested a review from wanghan-iapcm February 20, 2024 01:42
Copy link
Copy Markdown
Collaborator

@wanghan-iapcm wanghan-iapcm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wanghan-iapcm wanghan-iapcm added this pull request to the merge queue Feb 20, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 20, 2024
@njzjz njzjz added this pull request to the merge queue Feb 20, 2024
Merged via the queue into deepmodeling:devel with commit fe14402 Feb 20, 2024
@njzjz njzjz deleted the fix-neighbor-stat branch February 20, 2024 04:21
@njzjz njzjz mentioned this pull request Apr 2, 2024
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.

2 participants