[Scilab-users] circshift() : Scilab Enhancement Proposal

[antoine monmayrant]

Hi Samuel,


Thanks a lot for this nice SEP.
I have a couple of comments:

1) I think I should be good to also show some multidimensional examples 
using "shifts" and using both "shifts" and "dims". Sometimes in the 
scilab documentation, the user is served with examples only covering the 
most basic usage and not the advanced one with N optional arguments.

2) For multidimentional cshift, the order of the shifts (for example 
shift  by N along the lines than shift by M along the columns or the 
other way round) is key, right? Inverting the order would give a 
different result, right? Should this be specified in the SEP:

circshift(A, shifts) first circularly shifts indices of A rows by 
shifts(1), then the indices of A columns by shifts(2), then indices of A 
layers by shifts(3), etc.

Cheers,

Antoine

(& Happy New Year to all!)

Le 23/12/2017 à 19:06, Samuel Gougeon a écrit :
> Dear co-scilabers,
>
> After the wish reported at http://bugzilla.scilab.org/7293 in 2010,
> some cshift() function was proposed on the FileExchange at
> http://fileexchange.scilab.org/toolboxes/161000 to provide a way
> to circularly shift the rows or columns of a matrix or an hypermatrix.
>
> In order to include this basic but non-trivial feature in Scilab 6.1,
> a Scilab Enhancement Proposal is posted on review at
> https://codereview.scilab.org/#/c/19625/
>
> Proposed features are described in the help page in PDF format in 
> attachment.
>
> Every comment is welcome.
>
> Best regards
> Samuel Gougeon
>
>
>
> _______________________________________________
> users mailing list
> users at lists.scilab.org
> http://lists.scilab.org/mailman/listinfo/users

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.scilab.org/pipermail/users/attachments/20180103/2c6bf5a4/attachment.htm>