Skip to content

give pass actions different names #2142

Merged
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
njzjz:give-pass-different-names
Dec 1, 2022
Merged

give pass actions different names #2142
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
njzjz:give-pass-different-names

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Nov 30, 2022

Follow up #2141. They should have different names, otherwise I cannot set each of them in the setting.

Follow up deepmodeling#2141. They should have different names, otherwise I cannot set each of them in the setting.
@codecov-commenter
Copy link

codecov-commenter commented Nov 30, 2022

Codecov Report

Base: 74.12% // Head: 74.11% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (61c12c3) compared to base (c03e3e2).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #2142      +/-   ##
==========================================
- Coverage   74.12%   74.11%   -0.02%     
==========================================
  Files         201      201              
  Lines       19760    19760              
  Branches     1414     1414              
==========================================
- Hits        14648    14645       -3     
  Misses       4175     4175              
- Partials      937      940       +3     
Impacted Files Coverage Δ
source/lib/src/fmt_nlist.cc 78.31% <0.00%> (-1.21%) ⬇️
source/lib/include/ComputeDescriptor.h 76.65% <0.00%> (-0.45%) ⬇️

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 6721bc6 into deepmodeling:devel Dec 1, 2022
@njzjz njzjz deleted the give-pass-different-names branch December 1, 2022 06:03
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
Follow up deepmodeling#2141. They should have different names, otherwise I cannot
set each of them in the setting.
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.

3 participants