Skip to content

find protobuf headers in an automatic way#1975

Merged
wanghan-iapcm merged 2 commits into
deepmodeling:develfrom
njzjz:find-protobuf-headers
Oct 7, 2022
Merged

find protobuf headers in an automatic way#1975
wanghan-iapcm merged 2 commits into
deepmodeling:develfrom
njzjz:find-protobuf-headers

Conversation

@njzjz

@njzjz njzjz commented Oct 6, 2022

Copy link
Copy Markdown
Member

Fixes #1957.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Base: 75.71% // Head: 42.16% // Decreases project coverage by -33.54% ⚠️

Coverage data is based on head (a4fa2c2) compared to base (e9a00b4).
Patch has no changes to coverable lines.

❗ Current head a4fa2c2 differs from pull request most recent head 385c13e. Consider uploading reports for the commit 385c13e to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##            devel    #1975       +/-   ##
===========================================
- Coverage   75.71%   42.16%   -33.55%     
===========================================
  Files         191       66      -125     
  Lines       18695     6526    -12169     
  Branches      854        0      -854     
===========================================
- Hits        14154     2752    -11402     
+ Misses       4153     3774      -379     
+ Partials      388        0      -388     
Impacted Files Coverage Δ
source/op/soft_min_virial.cc 3.84% <0.00%> (-90.60%) ⬇️
source/op/soft_min_force.cc 5.00% <0.00%> (-87.86%) ⬇️
source/op/soft_min.cc 3.07% <0.00%> (-87.84%) ⬇️
source/op/pair_tab.cc 2.10% <0.00%> (-87.07%) ⬇️
source/op/ewald_recp.cc 2.77% <0.00%> (-86.51%) ⬇️
source/op/prod_env_mat_multi_device_nvnmd.cc 1.05% <0.00%> (-83.85%) ⬇️
source/op/prod_force.cc 3.77% <0.00%> (-78.14%) ⬇️
source/op/unaggregated_grad.cc 1.19% <0.00%> (-77.11%) ⬇️
source/op/neighbor_stat.cc 1.98% <0.00%> (-76.84%) ⬇️
source/op/prod_virial.cc 3.12% <0.00%> (-76.72%) ⬇️
... and 161 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.

@wanghan-iapcm wanghan-iapcm merged commit 6733a06 into deepmodeling:devel Oct 7, 2022
njzjz added a commit to njzjz/deepmd-kit that referenced this pull request Oct 7, 2022
Follow-up of deepmodeling#1975 to fix deepmodeling#1957.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
wanghan-iapcm pushed a commit that referenced this pull request Oct 8, 2022
Follow-up of #1975 to fix #1957.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
Fixes deepmodeling#1957.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
)

Follow-up of deepmodeling#1975 to fix deepmodeling#1957.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] How to specify TENSORFLOW_ROOT during install?

3 participants