wkcn commented on issue #9939: add multi proposal operator (cpu version) and 
fix the bug in proposal op (gpu version)
URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370320488
 
 
   For the cpu/gpu consistency, it's interesting that the number of valid 
anchors in the CPU implementation and the GPU implementation may be different.
   The reason is that the float precision in CPU's and GPU's is different.
   The size of some anchors may be near the margin of the minimal valid anchor 
or the maximal valid anchor.
   
   The margin of the minimal valid anchor:
   
https://github.com/wkcn/incubator-mxnet/blob/add_multi_proposal_cpu_version/src/operator/contrib/multi_proposal.cc#L159
   
   The margin of the maximal valid anchor.
   
https://github.com/wkcn/incubator-mxnet/blob/add_multi_proposal_cpu_version/src/operator/contrib/multi_proposal.cc#L89
   
   I want to create testing sample to avoid these margins.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to