-
Notifications
You must be signed in to change notification settings - Fork 542
I meet problem during implement light_head_rcnn #45
Comments
Hi, where did you take |
PSRoI_Align from https://github.com/zengarden/light_head_rcnn |
I'm pretty sure |
light head rcnn model also is not converge use PSRoI_Align from https://github.com/zengarden/light_head_rcnn ,I pull requests:#48 |
I will carefully check the code |
@Rizhiy could you share PSRoIPooling ? I compare the code with https://github.com/msracver/Deformable-ConvNets/blob/master/rfcn/operator_cxx/psroi_pooling.cu,the different as shown: |
@hewumars I haven't yet got PSRoIPooling to work in PyTorch either. |
@Rizhiy How is the PSROI pooling going? I have seen you in many different repos. I think we both focus on the light-head rcnn, right? I don't get the PSRoIpooling in Pytorch either. I think it could be easier to use the code from the official tf implementation. |
@YanShuo1992 I'm currently using roytseng-tw/Detectron.pytorch, so far I have focused on getting the best mAP, so didn't put much work in light-head. I will try to let you know if I get something working. |
That's not entirely correct. You need to pass output of res5, through a layer which has I suggest you check https://github.com/msracver/Deformable-ConvNets/blob/f4e163719c8e63cfad7af1caaaab93d373750393/rfcn/symbols/resnet_v1_101_rfcn.py#L785-L798 for reference. |
@Rizhiy So that I check the .cu code of psroipooling. I find you commit that do not use rounding in the roialign_kernel.cu. Can you tell me the reason for that or what problem it will lead? |
@YanShuo1992 |
@GYxiaOH |
In __init__() save self.num_data for use in __len__()
loss_bbox is not converge.other loss(loss_cls,loss_rpn_cls,loss_bbox) is converge.can I push the code to you for debug.
The text was updated successfully, but these errors were encountered: