-
Notifications
You must be signed in to change notification settings - Fork 86
Rename depth to channels #1118
Rename depth to channels #1118
Conversation
This PR needs Approvals as follows.
Please choose reviewers and requet reviews! Click to see how to approve each reviewsYou can approve this PR by triggered comments as follows.
See all trigger commentsPlease replace [Target] to review target
|
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.
RA (reverse-shadowing)
Please assign a readability reviewer after getting OA. |
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.
OA
/ready |
⏳Merge job is queued... |
😥This PR is not approved yet. |
RA |
/ready |
⏳Merge job is queued... |
|
/ready |
⏳Merge job is queued... |
😥Not mergeable. Please fix problems and send |
What this patch does to fix the issue.
there are several naming for 'number of channels in a layer:
depth
channel
channels
This PR unify to
channels
.kernel_depth (for convolution) -> input_channels
depths -> channels_list
channel -> channels
depth -> channels
Link to any relevant issues or pull requests.