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

Add Region Proposal Network for Faster R-CNN, including AnchorLayer, … #2828

Closed

Conversation

guoshengCS
Copy link
Contributor

Add Region Proposal Network for Faster R-CNN, including AnchorLayer, RPNLossLayer and ProposalLayer.

There are correspondences between RPN and SSD:

  • AnchorLayer corresponding to PriorboxLayer
  • RPNLossLayer corresponding to MultiBoxLossLayer
  • ProposalLayer corresponding to DetectionOutputLayer

initDataLayer(
configt, &dataLayers, &datas, &layerMap, "anchors", 1, false, use_gpu);

dataLayers[0]->getOutputValue()->copyFrom(*inputAnchor);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that the size of dataLayers[*]->outputvalue is wrong.
You can use LOG(INFO)<< "dataLayers[0] size: "<<dataLayers[0]->getOutputValue()->getElementCnt(); to check if the size of OuptValue equals to (or greater than?) size of inputAnchor.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. I have checked it as you said, and the result showed the same size. This unit test has passed.

@luotao1
Copy link
Contributor

luotao1 commented Feb 1, 2019

感谢您给PaddlePaddle贡献代码。由于Paddle V1/V2版本已不再维护,相关代码也已从develop分支上删除,因此关闭您的PR,欢迎您向Paddle最新版-Fluid贡献代码。
Thanks for contributing to PaddlePaddle! Since V1/V2 will not be maintained anymore, and related codes have been deleted from develop branch as well, we close this PR. Welcome to contribute to Fluid——the latest version of PaddlePaddle.

@luotao1 luotao1 closed this Feb 1, 2019
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

Successfully merging this pull request may close these issues.

None yet

3 participants