Re: [eigen] New indexing/slicing API: almost ready to be merged

[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]


awesome work :-)

why not neglecting the braceless overloads and only allowing braces which makes it a bit more consistent.
I dont think there is much of an overhead in putting all into brackets…

by the way what type is {all} ?  Is it an array of the type returned by „all“ with length 1?

cheers gabriel



> Am 12.01.2017 um 10:34 schrieb Massimiliano Janes <max.jns@xxxxxxxxx>:
> 
> 
>> On 11 Jan 2017, at 22:13, Gael Guennebaud <gael.guennebaud@xxxxxxxxx> wrote:
>> 
>> However I don't see how to generalize it to multi-dimensional tensors
> 
> a workaround could be requiring users to wrap *all* arguments into braces in this case:
> 
> A( {all}, {1,2,3} )
> A( {1,2,3}, { all }, {5,6,7}, { seq(2,4) } )
> 
> eventually providing braceless overloads only for <=2d
> 
> 
> 
> 



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/