I can't understand the diagrams you have here.  Can you try with a
```
code block?
```
It doesn't seem wrong to you that both removeRowsBy and removeColsBy both take a column vector?

I think removeColsBy should take a ColVector as the mask. I think removeRowsBy should take a RowVector as the mask.  I think we messed up with keepRowsBy.