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 2020/04/07 06:46:13 UTC

[GitHub] [incubator-mxnet] leezu opened a new issue #17990: Boolean indexing accesses out of bound elements

leezu opened a new issue #17990: Boolean indexing accesses out of bound elements
URL: https://github.com/apache/incubator-mxnet/issues/17990
 
 
   ## Description
   CI with updated toolchain (ie #17984) catches the bug.
   
   `vector: :_M_range_check: __n (which is 2) >= this->size() (which is 2)`
   
   ### Error Message
   ## To Reproduce
   Build with this simple patch
   
   ``` diff
   diff --git a/src/operator/numpy/np_boolean_mask_assign.cc b/src/operator/numpy/np_boolean_mask_assign.cc
   index e01ebb7c6..d5ab00835 100644
   --- a/src/operator/numpy/np_boolean_mask_assign.cc
   +++ b/src/operator/numpy/np_boolean_mask_assign.cc
   @@ -220,7 +220,7 @@ void NumpyBooleanAssignForwardCPU(const nnvm::NodeAttrs& attrs,
      // If there's no True in mask, return directly
      if (valid_num == 0) return;
   
   -  const TShape& vshape = inputs[2].shape_;
   +  const TShape& vshape = inputs.at(2).shape_;
   
      if (inputs.size() == 3U) {
        // tensor case
   ```
   
   OR follow the instructions in https://github.com/apache/incubator-mxnet/issues/17987 to trigger this via glibc assertions in debug build.
   
   CC: @haojin2  

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-mxnet] leezu commented on issue #17990: Boolean indexing accesses out of bound elements

Posted by GitBox <gi...@apache.org>.
leezu commented on issue #17990: Boolean indexing accesses out of bound elements
URL: https://github.com/apache/incubator-mxnet/issues/17990#issuecomment-614188033
 
 
   @haojin2 will you fix the bug and reenable the tests?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-mxnet] haojin2 commented on issue #17990: Boolean indexing accesses out of bound elements

Posted by GitBox <gi...@apache.org>.
haojin2 commented on issue #17990: Boolean indexing accesses out of bound elements
URL: https://github.com/apache/incubator-mxnet/issues/17990#issuecomment-614999441
 
 
   Fixed in #17796, test re-enabled

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-mxnet] yzhliu commented on issue #17990: Boolean indexing accesses out of bound elements

Posted by GitBox <gi...@apache.org>.
yzhliu commented on issue #17990:
URL: https://github.com/apache/incubator-mxnet/issues/17990#issuecomment-619663620


   should be fixed by #18158 


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [incubator-mxnet] yzhliu edited a comment on issue #17990: Boolean indexing accesses out of bound elements

Posted by GitBox <gi...@apache.org>.
yzhliu edited a comment on issue #17990:
URL: https://github.com/apache/incubator-mxnet/issues/17990#issuecomment-619663620


   should be fixed by #18158 (originally #17796)


----------------------------------------------------------------
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:
users@infra.apache.org