Kacper-Pietkun opened a new pull request, #21112:
URL: https://github.com/apache/incubator-mxnet/pull/21112

   ## Description ##
   _npi_repeats gave correct outputs only when repetition happened on 0th axis. 
That's why swapaxes function was called before and after triggering 
_npi_repeats operator. swapaxes() was called from python side, thus when 
_npi_repeats was a node in a computational graph it resulted in wrong output 
(swapaxes wasn't called in that case).
   
   ## Solution ##
   Trigger SwapAxis operator directrly by _npi_repeats operator.
   
   ## Checklist ##
   ### Essentials ###
   - [x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], 
[FEATURE], [DOC], etc)
   - [x] Changes are complete (i.e. I finished coding on this PR)
   - [x] All changes have test coverage
   - [x] Code is well-documented
   
   ## Comments ##
   - Added check for repeats parameter - it throws ValueError when length of 
the list is not equal to the number of elements on the axis that is going to be 
repeated (numpy compatibility)
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to