Conversation
davisking
reviewed
Feb 10, 2025
Owner
davisking
left a comment
There was a problem hiding this comment.
Nice yeah this seems like a useful thing to have :D
| const tensor& src, | ||
| size_t sk, size_t snr, size_t snc, | ||
| size_t k, size_t nr, size_t nc | ||
| ); |
Owner
There was a problem hiding this comment.
Add docs for this (like you see for the other functions in this file :) )
Contributor
Author
There was a problem hiding this comment.
Oh, I forgot that. Added the doc string. Thanks!
Owner
|
Thanks for the PR :) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I am training a U-Net using the architecture from U-Net: Convolutional Networks for Biomedical Image Segmentation. It requires cropping the tensors from the encoding stages to concatenate with the outputs of the decoding stages. I created the slice layer to implement this.
AFAICK, there are no layers in dlib that accommodate this. However, I probably should have asked before creating this one.
Some of my considerations:
copy_tensorto copy non-contiguous rows with a single kernel call.Thanks for taking a look!