Skip to content

bump lammps to stable_2Aug2023#2717

Merged
wanghan-iapcm merged 13 commits intodeepmodeling:develfrom
njzjz:lammps-stable_2Aug2023
Aug 7, 2023
Merged

bump lammps to stable_2Aug2023#2717
wanghan-iapcm merged 13 commits intodeepmodeling:develfrom
njzjz:lammps-stable_2Aug2023

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Aug 4, 2023

No description provided.

@github-actions github-actions bot added the Docs label Aug 4, 2023
@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (4fa54ec) 78.64% compared to head (300a02b) 78.65%.

Additional details and impacted files
@@           Coverage Diff           @@
##            devel    #2717   +/-   ##
=======================================
  Coverage   78.64%   78.65%           
=======================================
  Files         239      239           
  Lines       25465    25465           
  Branches     1517     1517           
=======================================
+ Hits        20028    20029    +1     
+ Misses       5045     5044    -1     
  Partials      392      392           

see 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions github-actions bot added the LAMMPS label Aug 4, 2023
@njzjz njzjz requested a review from Yi-FanLi August 4, 2023 20:22
Copy link
Collaborator

@Yi-FanLi Yi-FanLi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I'm surprised that I did not explicitly use atom ids to compare expected quantities and lammps's output in the past.

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