TaoLv commented on issue #16866: skip quantized conv flaky case
URL: https://github.com/apache/incubator-mxnet/pull/16866#issuecomment-559325269
 
 
   @ChaiBapchya @access2rohit @larroy Really appreciate your suggestions and 
frugal spirit for MXNet development. I think this PR is not the first time for 
contributors to use CI for debugging purpose, and most likely it will not be 
the last one. I just took two simple filters on the PR list:
   
https://github.com/apache/incubator-mxnet/pulls?utf8=%E2%9C%93&q=is%3Apr+in%3Atitle+%22do+not+merge%22
   
https://github.com/apache/incubator-mxnet/pulls?utf8=%E2%9C%93&q=is%3Apr+in%3Atitle+%22debug%22
   @access2rohit Can you explain your PRs at the top of the filtered list? If 
you really worry about the usage of CI resources, could you please bring this 
topic to @dev to forbid this behavior and note it somewhere if you get a 
consensus there?
   
   I believe @xinyu-intel has tried his best to reproduce this issue on both 
local machine and also on EC2 instances to better emulate the CI environment. 
But unfortunately, he cannot reproduce the error and I think it's not the first 
time that we cannot reproduce an error out of CI.
   
   Another choice is maybe someone can contribute to the CI system so 
developers can only trigger just one test suits rather than all of the 11 tests 
suits. It will be very helpful especially to document contributors.

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


With regards,
Apache Git Services

Reply via email to