Skip to content

error if LAMMPS_VERSION_NUMBER is not defined#1849

Merged
wanghan-iapcm merged 1 commit into
deepmodeling:develfrom
njzjz:error-for-LAMMPS_VERSION_NUMBER
Aug 7, 2022
Merged

error if LAMMPS_VERSION_NUMBER is not defined#1849
wanghan-iapcm merged 1 commit into
deepmodeling:develfrom
njzjz:error-for-LAMMPS_VERSION_NUMBER

Conversation

@njzjz

@njzjz njzjz commented Aug 7, 2022

Copy link
Copy Markdown
Member

Throw an error if some one (like @Yi-FanLi) compiles LAMMPS with USER-DEEPMD using CMAKE and does not define LAMMPS_VERSION_NUMBER.

Ref: https://stackoverflow.com/a/2221535/9567349

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@github-actions github-actions Bot added the LAMMPS label Aug 7, 2022
@codecov-commenter

codecov-commenter commented Aug 7, 2022

Copy link
Copy Markdown

Codecov Report

Merging #1849 (ff38b87) into devel (0010a7e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            devel    #1849   +/-   ##
=======================================
  Coverage   76.98%   76.98%           
=======================================
  Files         117      117           
  Lines        9417     9417           
=======================================
  Hits         7250     7250           
  Misses       2167     2167           

Help us with your feedback. Take ten seconds to tell us how you rate us.

@wanghan-iapcm wanghan-iapcm merged commit aec488b into deepmodeling:devel Aug 7, 2022
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
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