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-370447850 @marcoabreu The reason to add USE_STABLE_SORT_FOR_PROPOSAL flag is for the cpu/gpu consistency test. Proposal Operator (CPU) uses unstable sort to sort the anchors by scores, however Proposal Operator (GPU) uses stable sort. It leads to different outputs even if the code is right. For the performance, I couldn't replace unstable sort with stable sort in Proposal (CPU). And for the cpu/gpu consistency test, I think it's necessary to use a building flag to test these operators.
---------------------------------------------------------------- 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
