-
Notifications
You must be signed in to change notification settings - Fork 220
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
I have implement your pmtd ideas, but the result is not good enough. #2
Comments
Could you provide your implementation details of PMTD and the evaluation score of ICDAR 2017?I'm afraid that I can't help you without these details. Train StageTrain SchedulerWe train PMTD use 32 (noted as gpu_num) TITIANX 12G GPUs with SyncBatchNorm.
The learning rate changes as follows:
Loss Design
note: RPN Loss and Bounding Box LossRPN Loss and Bounding Box Loss are same with Mask RCNN, only changing the Mask Loss:
note: Data augmentation, RPN Anchor and OHEMthese details should implement as the paper described. Remember to random resize the image without keeping the aspect ratio
Pyramid Label generation:
Test Stagejust follow the release code |
thank you for your response, I have few ops difference with your proposal.
|
Hi @JingChaoLiu , I have another question.
|
Yes, we use them. The boxes with |
Hi @tsing-cv , could you possibly share your implementation. I would appreciate it |
@GarrettLee My work fell short of what the paper claimed |
Hi @JingChaoLiu , do you mean that |
5 * l1_loss(predict_tensor, target_tensor) |
Hi @JingChaoLiu ,
As for the targets everything is ok (inspected visually). Is there anything I should tweak to make this loss converge? Thanks in advance for any suggestions Regards, |
I found I had also PMTD mask predictor "MaskRCNNC4Predictor_Upsample". When I switched back to "MaskRCNNC4Predictor" mask loss seems alright now :) |
In plain maskrcnn-benchmark, the mask loss is calculated by 'binary_cross_entropy', which is implemented by 'mask.sigmoid() + dot product'. When trained with pyramid label, append |
Maybe we append a sigmoid() function inside the |
@JingChaoLiu, thank you for pointing this out to me. Indeed there's |
My plan for now is to:
Any other suggestions? :) |
Same issues. Any solutions? |
请问有可用的训练代码参考吗? |
No description provided.
The text was updated successfully, but these errors were encountered: