Skip to content

fix bugs of dipole charge modifier: binary str and missing frozen nodes#1124

Merged
amcadmus merged 1 commit intodeepmodeling:develfrom
amcadmus:fix-datamodi
Sep 10, 2021
Merged

fix bugs of dipole charge modifier: binary str and missing frozen nodes#1124
amcadmus merged 1 commit intodeepmodeling:develfrom
amcadmus:fix-datamodi

Conversation

@amcadmus
Copy link
Member

No description provided.

@amcadmus amcadmus requested a review from njzjz September 10, 2021 06:21
@codecov-commenter
Copy link

codecov-commenter commented Sep 10, 2021

Codecov Report

Merging #1124 (796d934) into devel (ed36276) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel    #1124      +/-   ##
==========================================
- Coverage   75.48%   75.47%   -0.02%     
==========================================
  Files          88       88              
  Lines        7042     7043       +1     
==========================================
  Hits         5316     5316              
- Misses       1726     1727       +1     
Impacted Files Coverage Δ
deepmd/entrypoints/freeze.py 73.58% <ø> (ø)
deepmd/infer/deep_pot.py 68.55% <0.00%> (-0.44%) ⬇️

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 ed36276...796d934. Read the comment docs.

@amcadmus amcadmus merged commit 1526587 into deepmodeling:devel Sep 10, 2021
@amcadmus amcadmus deleted the fix-datamodi branch September 10, 2021 11:50
njzjz added a commit to njzjz/deepmd-kit that referenced this pull request Sep 21, 2023
- add pre-commit config, including black and common fixes.
- run pre-commit.
- manually fix a YAML file.
- skip check json and broken symlinks.
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.

3 participants