Skip to content

resolve macos clang errors#1993

Merged
wanghan-iapcm merged 6 commits into
deepmodeling:develfrom
njzjz:link-protobuf
Oct 14, 2022
Merged

resolve macos clang errors#1993
wanghan-iapcm merged 6 commits into
deepmodeling:develfrom
njzjz:link-protobuf

Conversation

@njzjz

@njzjz njzjz commented Oct 12, 2022

Copy link
Copy Markdown
Member

This is also aimed at resolving macos clang errors:

  1. link to protobuf if it's used and separated;
  2. add link flags to lmp lib.
  3. set rpath; remove gnu specific flags;

@github-actions github-actions Bot added the OP label Oct 12, 2022
@codecov-commenter

codecov-commenter commented Oct 12, 2022

Copy link
Copy Markdown

Codecov Report

Base: 73.56% // Head: 19.11% // Decreases project coverage by -54.44% ⚠️

Coverage data is based on head (c6d7313) compared to base (77c7ad4).
Patch has no changes to coverable lines.

❗ Current head c6d7313 differs from pull request most recent head 3e8c2b1. Consider uploading reports for the commit 3e8c2b1 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##            devel    #1993       +/-   ##
===========================================
- Coverage   73.56%   19.11%   -54.45%     
===========================================
  Files         198      196        -2     
  Lines       19916    18169     -1747     
  Branches      854        0      -854     
===========================================
- Hits        14651     3473    -11178     
- Misses       4877    14696     +9819     
+ Partials      388        0      -388     
Impacted Files Coverage Δ
deepmd/utils/convert.py 19.44% <ø> (+0.47%) ⬆️
source/api_cc/src/AtomMap.cc 100.00% <ø> (ø)
source/lib/src/gelu.cc 0.00% <0.00%> (-100.00%) ⬇️
source/lib/src/region.cc 0.00% <0.00%> (-100.00%) ⬇️
source/lib/src/map_aparam.cc 0.00% <0.00%> (-100.00%) ⬇️
source/lib/src/env_mat_nvnmd.cc 0.00% <0.00%> (-100.00%) ⬇️
source/lib/src/soft_min_switch.cc 0.00% <0.00%> (-100.00%) ⬇️
source/lib/src/soft_min_switch_force.cc 0.00% <0.00%> (-100.00%) ⬇️
source/lib/src/soft_min_switch_virial.cc 0.00% <0.00%> (-100.00%) ⬇️
source/lib/src/soft_min_switch_force_grad.cc 0.00% <0.00%> (-100.00%) ⬇️
... and 146 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.

@njzjz njzjz marked this pull request as draft October 12, 2022 20:43
@github-actions github-actions Bot added the C++ label Oct 12, 2022
@njzjz njzjz changed the title op_abi: link to protobuf if it's seperated link to protobuf if it's seperated Oct 12, 2022
@njzjz njzjz changed the title link to protobuf if it's seperated link to protobuf if it's used and seperated Oct 12, 2022
@njzjz njzjz changed the title link to protobuf if it's used and seperated resolve macos clang errors Oct 13, 2022
@github-actions github-actions Bot added the i-PI label Oct 13, 2022
@njzjz njzjz marked this pull request as ready for review October 13, 2022 22:35
@njzjz njzjz requested review from wanghan-iapcm and removed request for wanghan-iapcm October 13, 2022 22:35
@njzjz njzjz marked this pull request as draft October 13, 2022 22:39
@njzjz njzjz requested a review from wanghan-iapcm October 13, 2022 22:42
@njzjz njzjz marked this pull request as ready for review October 13, 2022 22:42
@wanghan-iapcm wanghan-iapcm merged commit 27ca650 into deepmodeling:devel Oct 14, 2022
@njzjz njzjz deleted the link-protobuf branch October 14, 2022 03:15
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
This is also aimed at resolving macos clang errors:
1. link to protobuf if it's used and separated;
2. add link flags to lmp lib.
3. set rpath; remove gnu specific flags;

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants