Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

About CropSplit and CropSplitGt in mmdet.ops #44

Open
ztt0821 opened this issue Mar 29, 2021 · 4 comments
Open

About CropSplit and CropSplitGt in mmdet.ops #44

ztt0821 opened this issue Mar 29, 2021 · 4 comments

Comments

@ztt0821
Copy link

ztt0821 commented Mar 29, 2021

Hi, When I use your code, I found one question about CropSplit and CropSplitGt in mmdet.ops. May I ask the function of the CropSplit. I found that you define self.crop_cuda = CropSplit(2) and self.crop_gt_cuda = CropSplitGt(2) at first, so what's the meaning of "2"? I have print the input feature map size of pred_masks = self.crop_cuda(pred_masks, bbox_dt), which is "[4,288,288, number of coefficient]", and output size is [288,288,number of coefficient]. May I ask what have done in CropSplit?

@ztt0821
Copy link
Author

ztt0821 commented Mar 29, 2021

I also found the input size of gt_mask_crop = self.crop_gt_cuda(gt_mask_new, bbox_dt) is [288, 288, number of coefficient], so does self.crop_cuda merge the 4 parts ?

@JialeCao001
Copy link
Owner

JialeCao001 commented Mar 29, 2021

@ztt0821 Yeah. These functions aim at combining 4 parts in cuda.

@ztt0821
Copy link
Author

ztt0821 commented Mar 29, 2021

@ztt0821 Yeah. These functions aim at combining 4 parts in cuda.

Thanks for your reply. May I ask what is the meaning of "2" in self.crop_cuda = CropSplit(2)? Does pred_masks = self.crop_cuda(pred_masks, bbox_dt) also crop the segmentation mask based on box location?

@JialeCao001
Copy link
Owner

JialeCao001 commented Mar 29, 2021

@ztt0821 It demonstrates that the regions of detected bounding box are split into 2x2 sub-regions for mask prediction. Yes, it crops the segmentation mask.

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

No branches or pull requests

2 participants