Skip to content

fix the header of "lr"#2462

Merged
wanghan-iapcm merged 2 commits intodeepmodeling:develfrom
njzjz:fix-lr-header
Apr 19, 2023
Merged

fix the header of "lr"#2462
wanghan-iapcm merged 2 commits intodeepmodeling:develfrom
njzjz:fix-lr-header

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Apr 17, 2023

xref: #2348

xref: deepmodeling#1142

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz
Copy link
Member Author

njzjz commented Apr 17, 2023

cc @HuangJiameng

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

codecov bot commented Apr 17, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (69e712c) 74.40% compared to head (7b388f5) 74.40%.

Additional details and impacted files
@@           Coverage Diff           @@
##            devel    #2462   +/-   ##
=======================================
  Coverage   74.40%   74.40%           
=======================================
  Files         227      227           
  Lines       23394    23394           
  Branches     1673     1673           
=======================================
  Hits        17407    17407           
+ Misses       4891     4890    -1     
- Partials     1096     1097    +1     
Impacted Files Coverage Δ
deepmd/train/trainer.py 81.83% <100.00%> (ø)

... and 2 files with indirect coverage changes

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 in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@njzjz njzjz requested a review from wanghan-iapcm April 17, 2023 21:38
@wanghan-iapcm wanghan-iapcm merged commit 9a3ac96 into deepmodeling:devel Apr 19, 2023
hztttt pushed a commit to hztttt/deepmd-kit that referenced this pull request Apr 21, 2023
xref: deepmodeling#2348

---------

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants