jwyang / jwyang/fpn.pytorch

Your code doesn't match your comment.

Open
#2 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
969
Forks
220
PR merge metrics
No merged PRs in 30d

Description

In your code, you note that "the original paper used pool_size = 7 for cls branch, and 14 for mask branch, to save the computation time, we first use 14 as the pool_size, and then do stride=2 pooling for cls branch." (37th line in fpn.py file).
But according to your default configuration file (config.py in folder utils), the cfg.POOLING_SIZE is still 7, not 14.
And there's not a "stride=2 pooling function" in the forward function of your _FPN class, although I can find a function called RCNN_roi_feat_ds in resnet.py which might be created for it but is a convolutional function, not a pooling one and not in use.
Also there's a comment in 235th line of fpn.py, indicating that the size of the output should be 14 * 14. But actually it is 7 * 7.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.