Skip to content

move AutoBatchSize from dp test to DeepPot.eval#1173

Merged
amcadmus merged 4 commits intodeepmodeling:develfrom
njzjz:autobatchsize
Sep 28, 2021
Merged

move AutoBatchSize from dp test to DeepPot.eval#1173
amcadmus merged 4 commits intodeepmodeling:develfrom
njzjz:autobatchsize

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Sep 26, 2021

And reverts #1154 as calc_model_devi also calls dp.eval.

@njzjz njzjz changed the title move AutoBatchSize from dp test to dp.eval move AutoBatchSize from dp test to DeepPot.eval Sep 26, 2021
@njzjz njzjz requested a review from amcadmus September 26, 2021 07:32
@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2021

Codecov Report

Merging #1173 (9721064) into devel (d8acbb8) will decrease coverage by 0.02%.
The diff coverage is 53.12%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel    #1173      +/-   ##
==========================================
- Coverage   76.06%   76.04%   -0.03%     
==========================================
  Files          91       91              
  Lines        7241     7258      +17     
==========================================
+ Hits         5508     5519      +11     
- Misses       1733     1739       +6     
Impacted Files Coverage Δ
deepmd/entrypoints/test.py 11.53% <0.00%> (-0.37%) ⬇️
deepmd/infer/model_devi.py 53.01% <11.11%> (+0.57%) ⬆️
deepmd/infer/deep_eval.py 92.45% <54.54%> (-4.43%) ⬇️
deepmd/infer/deep_pot.py 68.86% <90.00%> (+0.30%) ⬆️
deepmd/utils/batch_size.py 96.00% <100.00%> (ø)

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 d8acbb8...9721064. Read the comment docs.

@njzjz njzjz removed the request for review from amcadmus September 26, 2021 10:39
@njzjz njzjz marked this pull request as draft September 26, 2021 10:39
@njzjz njzjz marked this pull request as ready for review September 26, 2021 14:20
@njzjz njzjz requested a review from Ericwang6 September 26, 2021 23:36
@amcadmus amcadmus merged commit 5843c02 into deepmodeling:devel Sep 28, 2021
njzjz pushed a commit to njzjz/deepmd-kit that referenced this pull request Sep 21, 2023
<!--pre-commit.ci start-->
updates:
- [github.com/psf/black: 23.1.0 →
23.3.0](psf/black@23.1.0...23.3.0)
- [github.com/charliermarsh/ruff-pre-commit: v0.0.259 →
v0.0.260](astral-sh/ruff-pre-commit@v0.0.259...v0.0.260)
<!--pre-commit.ci end-->

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

4 participants