You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2022/05/04 16:17:14 UTC

[GitHub] [incubator-mxnet] DominikaJedynak opened a new pull request, #21016: [v1.x] Added possibility to re-run tests with all batches

DominikaJedynak opened a new pull request, #21016:
URL: https://github.com/apache/incubator-mxnet/pull/21016

   ## Description ##
   Change analogous to https://github.com/apache/incubator-mxnet/pull/21008 on master.
   
   This PR adds the possibility to re-run the test with all batches used for calibration of quantized model.
   If test fails with only one batch used for calibration, it will be re-run with all batches to achieve better representation of the whole data range and more precise quantization scales. Thanks to that, loss in accuracy caused by too small calibration dataset will be avoided.
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@mxnet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-mxnet] DominikaJedynak closed pull request #21016: [v1.x] Added possibility to re-run tests with all batches for calibration

Posted by GitBox <gi...@apache.org>.
DominikaJedynak closed pull request #21016: [v1.x] Added possibility to re-run tests with all batches for calibration
URL: https://github.com/apache/incubator-mxnet/pull/21016


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@mxnet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #21016: [v1.x] Added possibility to re-run tests with all batches

Posted by GitBox <gi...@apache.org>.
mxnet-bot commented on PR #21016:
URL: https://github.com/apache/incubator-mxnet/pull/21016#issuecomment-1117547250

   Hey @DominikaJedynak , Thanks for submitting the PR 
   All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands: 
   - To trigger all jobs: @mxnet-bot run ci [all] 
   - To trigger specific jobs: @mxnet-bot run ci [job1, job2] 
   *** 
   **CI supported jobs**: [website, clang, centos-gpu, sanity, windows-gpu, edge, miscellaneous, unix-gpu, windows-cpu, centos-cpu, unix-cpu]
   *** 
   _Note_: 
    Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin. 
   All CI tests must pass before the PR can be merged. 
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@mxnet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-mxnet] DominikaJedynak commented on pull request #21016: [v1.x] Added possibility to re-run tests with all batches for calibration

Posted by GitBox <gi...@apache.org>.
DominikaJedynak commented on PR #21016:
URL: https://github.com/apache/incubator-mxnet/pull/21016#issuecomment-1129696279

   These changes do not need to be incorporated because as long as all the quantization tests use one batch data only, the problem witch unprecise calibration scales which was occurring on master branch does not occur on v1x.
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@mxnet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org