ptrendx opened a new pull request #13911: Fix for test always returning true
URL: https://github.com/apache/incubator-mxnet/pull/13911
 
 
   ## Description ##
   Simple change to actually test the value of `a.dtype` in 
`test_randint_without_dtype`. Without this, nosetests complains: 
   ```
   /opt/mxnet/tests/python/gpu/../unittest/test_random.py:884: SyntaxWarning: 
assertion is always true, perhaps remove parentheses?
     assert(a.dtype, 'int32')
   ```
   
   ## Checklist ##
   ### Essentials ###
   Please feel free to remove inapplicable items for your PR.
   - [x] Changes are complete (i.e. I finished coding on this PR)
   - [x] To the my best knowledge, examples are either not affected by this 
change, or have been fixed to be compatible with this change
   

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

Reply via email to