Skip to content

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Feb 8, 2024

Fix #3118.

Note: head is not supported yet in DeepEval

Note: head is not supported yet in DeepEval
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@github-actions github-actions bot added the Python label Feb 8, 2024
if len(all_sys) == 0:
raise RuntimeError("Did not find valid system")
err_coll = []
siz_coll = []

Check notice

Code scanning / CodeQL

Unused local variable

Variable siz_coll is not used.
ae = ret[3]
av = ret[4]
ae = ae.reshape([numb_test, -1])
av = av.reshape([numb_test, -1])

Check notice

Code scanning / CodeQL

Unused local variable

Variable av is not used.
@codecov
Copy link

codecov bot commented Feb 9, 2024

Codecov Report

Attention: 162 lines in your changes are missing coverage. Please review.

Comparison is base (c235099) 75.19% compared to head (e058e3f) 75.19%.

Files Patch % Lines
deepmd/entrypoints/test.py 56.28% 160 Missing ⚠️
deepmd/pt/entrypoints/main.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #3249      +/-   ##
==========================================
- Coverage   75.19%   75.19%   -0.01%     
==========================================
  Files         372      373       +1     
  Lines       33136    33135       -1     
  Branches     1608     1608              
==========================================
- Hits        24916    24915       -1     
  Misses       7347     7347              
  Partials      873      873              

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

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz requested review from iProzd and wanghan-iapcm February 9, 2024 00:20
@njzjz njzjz changed the title global dp test backward-indepedent dp test Feb 9, 2024
@njzjz njzjz changed the title backward-indepedent dp test backend-indepedent dp test Feb 9, 2024
@wanghan-iapcm wanghan-iapcm merged commit 9a8f712 into deepmodeling:devel Feb 9, 2024
@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.

3 participants