Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

backend-indepedent dp test #3249

Merged
merged 2 commits into from
Feb 9, 2024
Merged

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 Note test

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 Note test

Variable av is not used.
deepmd/entrypoints/test.py Fixed Show fixed Hide fixed
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 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
46 checks passed
@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