sleepy-owl opened a new pull request #17760: Optimizing tests in test_loss.py
URL: https://github.com/apache/incubator-mxnet/pull/17760
 
 
   ## Description ##
   Changing some parameters in test_loss to reduce the running time of tests
   
   ## Checklist ##
   ### Essentials ###
   Please feel free to remove inapplicable items for your PR.
   - [ ] Changes are complete 
   
   ### Changes ###
   - [ ] Updated `test_bce_loss_with_pos_weight` and `test_ctc_loss_train`. 
Reduces the run time of each test from >2 seconds to less than 1.3 seconds 
locally. Tests pass with multiple seeds (>30).
   
   ## Comments ##
   - There are multiple opportunities for reducing the running time of tests by 
tuning some parameters like iterations, learning rate, etc.
   - For these changes, I verified that the tests still pass with multiple seeds
   - Would you guys be interested in such optimizations? If yes, I can help 
contribute more and look at optimizing other *more expensive* tests in the test 
suite. Please let me know if you have any other suggestions or any necessary 
checks that I should do on these tests.
   
   Thanks!
   
   
   

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