IMO, `Changes` section is not the place to explain the rationale of a PR but 
the place to list the specific changes included in the PR, such as "added a 
parameter X to function Y", "added a unit test in module Z" etc. Nonetheless, I 
am still not clear why "Fix the inconsistency between CPU & GPU implementation" 
would fix the flakiness in test. Please elaborate more in the `Description` 
section. It not only helps reviewers understand the PR but also helps other 
developers to learn from this fix. Thank you!

[ Full content available at: 
https://github.com/apache/incubator-mxnet/pull/12557 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to