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-369773050
   @piiswrong Yes, I will add it.
   I found Proposal OP (CPU implementation) uses **unstable sort** 
   and Proposal OP (GPU implementation) uses **stable sort** 
   And the accuracy of computation between CPU and GPU is different, it causes 
that NMS selects different boxes between CPU and GPU.
   When I replace `std::sort` with `std::stable_sort` in the CPU 
implementation, the cpu/gpu consistency test is passed.

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:

With regards,
Apache Git Services

Reply via email to