Skip to content

c: add DipoleChargeModifier #2204

Merged
wanghan-iapcm merged 13 commits into
deepmodeling:develfrom
njzjz:comment-modifier
Dec 30, 2022
Merged

c: add DipoleChargeModifier #2204
wanghan-iapcm merged 13 commits into
deepmodeling:develfrom
njzjz:comment-modifier

Conversation

@njzjz

@njzjz njzjz commented Dec 28, 2022

Copy link
Copy Markdown
Member

Add DipoleChargeModifier to C API.

In addition, this patch also contains the following changes:

  • fix bugs in deepmd::DeepTensor::DeepTensor and deepmd::DipoleChargeModifier::DipoleChargeModifier
  • add comments to deepmd::DipoleChargeModifier
  • support gpu_rank and other parameters for deepmd::hpp::DeepPot and deepmd::hpp::DeepTensor

njzjz added 12 commits December 27, 2022 15:38
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
This reverts commit 26094ca.
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@codecov-commenter

codecov-commenter commented Dec 28, 2022

Copy link
Copy Markdown

Codecov Report

Base: 74.86% // Head: 74.85% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (9da0e5a) compared to base (9749fb3).
Patch coverage: 79.51% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #2204      +/-   ##
==========================================
- Coverage   74.86%   74.85%   -0.02%     
==========================================
  Files         202      203       +1     
  Lines       20011    20096      +85     
  Branches     1433     1438       +5     
==========================================
+ Hits        14982    15043      +61     
- Misses       4129     4146      +17     
- Partials      900      907       +7     
Impacted Files Coverage Δ
source/api_cc/include/DataModifier.h 25.00% <ø> (ø)
source/api_cc/include/DeepTensor.h 83.33% <ø> (-16.67%) ⬇️
source/api_c/src/c_api.cc 80.99% <72.54%> (-4.88%) ⬇️
source/api_c/include/deepmd.hpp 93.63% <90.00%> (-0.37%) ⬇️
source/api_cc/src/DataModifier.cc 80.41% <100.00%> (+3.49%) ⬆️
source/api_cc/src/DeepTensor.cc 80.47% <100.00%> (ø)
source/lib/include/ComputeDescriptor.h 76.65% <0.00%> (-0.67%) ⬇️
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Comment thread source/api_cc/include/DataModifier.h Outdated
Comment thread source/api_cc/include/DataModifier.h Outdated
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz requested a review from wanghan-iapcm December 29, 2022 04:13
@wanghan-iapcm wanghan-iapcm merged commit 41180ca into deepmodeling:devel Dec 30, 2022
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
Add `DipoleChargeModifier` to C API.

In addition, this patch also contains the following changes:
* fix bugs in `deepmd::DeepTensor::DeepTensor` and
`deepmd::DipoleChargeModifier::DipoleChargeModifier`
* add comments to `deepmd::DipoleChargeModifier`
* support `gpu_rank` and other parameters for `deepmd::hpp::DeepPot` and
`deepmd::hpp::DeepTensor`

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
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