-
Notifications
You must be signed in to change notification settings - Fork 526
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pd: add CPP inference with LAMMPS #4467
Open
HydrogenSulfate
wants to merge
56
commits into
deepmodeling:devel
Choose a base branch
from
HydrogenSulfate:add_cpp_infer
base: devel
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,547
−13
Open
Changes from 3 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
bea3123
add LAMMPS CPP inference
HydrogenSulfate 2e6cae4
update code
HydrogenSulfate a8145fa
update CPP inference test(deeppot_sea)
HydrogenSulfate 0e6172d
refine code
HydrogenSulfate 79f0b98
fix suggestions from code rabbit
HydrogenSulfate c2a9295
update
HydrogenSulfate 5a048c4
update test shell
HydrogenSulfate 8800bb0
change createNlistTensor to createNlistTensorPD to avoid mutliple def…
HydrogenSulfate 269e96e
use CPU inference lib instead of GPU inference lib
HydrogenSulfate 3f7e5af
resize output vector size before copy to CPU
HydrogenSulfate ff50600
fix test_deeppot_pd related code
HydrogenSulfate c24cee2
update CMAKE to avoid lib not found
HydrogenSulfate ee6547e
Merge branch 'devel' into add_cpp_infer
HydrogenSulfate 2a1be37
update LD_LIBRARY_PATH
HydrogenSulfate ca60d08
move LD_LIBRARY_PATH to the begin position
HydrogenSulfate e26ca2a
update test_cc_local
HydrogenSulfate 64ccf14
add paddle inference lib into 'LD_LIBRARY_PATH' in test_cc
HydrogenSulfate 5470ce0
fix 29814 typo
HydrogenSulfate aaf3131
update mkl and onednn lib in test_cc
HydrogenSulfate a24a6e9
output more details in fsanitize
HydrogenSulfate 7e6f5f4
test for line number
HydrogenSulfate 7030ccc
add fsanitize-ignorelist for some mem leak can be ignored currently
HydrogenSulfate 94793f3
Merge branch 'devel' into add_cpp_infer
HydrogenSulfate 0e22018
fix fsanitize-ignorelist path
HydrogenSulfate 4e7183f
use $ instead of 29814
HydrogenSulfate c3a2b27
Update .github/workflows/test_cc.yml
HydrogenSulfate 0045f77
try format
njzjz d598c3e
use the clang compiler
njzjz 5b663f1
enable more C++ test case
HydrogenSulfate cbbd11e
fix all serialization and inference code
HydrogenSulfate 9f558e7
Merge branch 'add_cpp_infer' of https://github.com/HydrogenSulfate/de…
HydrogenSulfate a749056
restore gpu information printing code
HydrogenSulfate f48ba9b
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 33219f1
solve mutiple definition of omp functions
HydrogenSulfate 4801b2a
Merge branch 'add_cpp_infer' of https://github.com/HydrogenSulfate/de…
HydrogenSulfate f82aa99
fix dtype for inference model
HydrogenSulfate 196f821
Merge branch 'devel' into add_cpp_infer
HydrogenSulfate 76e9019
split pd C++ test from test_cc_local
HydrogenSulfate 8e85649
split pd C++ test from test_cc_local
HydrogenSulfate 2204844
export model in prim mode to bypass latent error in PNormGradKernel
HydrogenSulfate 26f8010
Merge branch 'add_cpp_infer' of https://github.com/HydrogenSulfate/de…
HydrogenSulfate 60dabca
add libpaddle_inference.so as module to igorelist.txt
HydrogenSulfate 17b39c1
use LSAN_OPTIONS instead of fsanitize-ignorelist
HydrogenSulfate 1679314
remove LD_LIBRARY_PATH in cov test env
HydrogenSulfate 2e9bfa7
Merge branch 'devel' into add_cpp_infer
HydrogenSulfate 2944d58
fix
HydrogenSulfate 8e5a9f7
update source/lmp/plugin/CMakeLists.txt
HydrogenSulfate 8940bd9
append original LD_LIBRARY_PATH to
HydrogenSulfate 4bc5db3
copy paddle so instead of set LD_LIBRARY_PATH
HydrogenSulfate b761e83
revert Clang to GCC
njzjz 38904c3
fix rtol to 1e-6 and atol to 1e-9
HydrogenSulfate 58711c7
Merge branch 'add_cpp_infer' of https://github.com/HydrogenSulfate/de…
HydrogenSulfate d265094
update rtol and atol for python unitest
HydrogenSulfate 05759cc
skip test_pair_deepmd_mpi
HydrogenSulfate 50122a7
Update build_cxx.sh
HydrogenSulfate 2a2fd1b
Fix PATH export location in test workflow
njzjz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Improve robustness of Paddle library download and configuration.
Several improvements could make the Paddle integration more robust:
Consider implementing these improvements:
Also, consider making third-party library paths configurable: