|Re: [eigen] On a flexible API for submatrices, slicing, indexing, masking, etc.|
[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]
We may or may not want support static indices. If we do, i and j can be supplied as compile time constants using the user suffix syntax, e.g., 3_c, as you described before. Alternatively, we can have the rule that indices can be supplied as template parameters to span. Consider:V(span<3, 6>) vs V(span(3_c, 6_c))V(span<3, 6>(step(s)) vs V(span(3_c, 6_c, step(s)))V(span<3, 6>(step<2>)) vs V(span(3_c, 6_c, step<2>))V(span<4>(len(n)) vs V(span(4_c, len(n))V(span<6>(len<8>, step(s))) vs V(span(6_c, len<8>, step(s));All except the first row look too noisy to me, so maybe we should just not support static indices...
In a parallel note, I would love for people to brainstorm more alternative names for range and span, since those are just about used everywhere, and the convention is all over the place. For example, armadillo's span takes in (first, last) instead of (first, len), and boost::range has an irange(first, last) function for integer range, and gsl::span takes in (first, len).Yes, that's the main purpose of this thread and wiki-page!gaeland forget about the "iota"-based API. What remains unclear to me is how to expose this compact 'c' function. I also have to generalize the proof-of concept demo to be sure that we can generalize it to multi-dimensional tensors.gaelI'm currently experimenting with an API like:range(start,stop); // step==1range(start,stop,step); // run-time steprange(start,stop,c<STEP>); // compile-time stepspan(start,len); // step==1span(start,len,step); // run-time stepspan(start,len,c<STEP>); // compile-time stepspan(start,c<LEN>); // compile-time length and step==1span(start,c<LEN>,c<STEP>); // compile-time length and stepspan(start,c<LEN>,step); // compile-time length and runtime stepAnd the usage remains the same, e.g.:B = A(range(...), span(...));Some remarks:The key advantage here is that the argument order never change! For the "range" case, it would be ok to write range<STEP>(start,stop), but for the "span" case since the length needs also to be defined at compile-time this is unmanageable.Another advantage compared to the demo on the wiki is that the "bounds-based" and "length-based" variants are similar, no odd API like the iota(len) stuff... Of course, this is also a drawback because there might be some naming confusions between 'range' versus 'span'. It might not be 100% obvious that one is based on 'bounds' and the other on a 'length', but here is the rationale:- 'range' is (for me) more related to the notions of interval, limits, gamut, etc. that are naturally defined by their 'bounds'.- 'span' is related to the notion of period of time, distance, width, extent, etc. and thus the notion of 'length' here.Compared to the demo on the wiki page, here the 'step' is moved to the last argument. This is not matlab friendly, but in c/c++ optional arguments go last, so this makes more sense.Another issue is that this approach is very compact only if we accept to define a Eigen::c and that the user import it in its current scope and use c++14 (perhaps c++11 with Yuanchen trick?). Otherwise it can become as verbose and unreadable as:Eigen::span(start, Eigen::Index_c<LEN>(), Eigen::Index_c<STEP>())Finally, we also have to decide whether the 'stop' argument should be an inclusive or an exclusive upper bound... To figure this out, I'll prepare a set of examples to see what's the most convenient. My intuition is that even though we are used of the STL's exclusive 'end', an inclusive upper bound would be more symmetric with the inclusive lower bound, and thus indexing from the end should be easier...OK, one more: with this approach we could easily enable compile-time start/stop with range to figure out the length at compile time: range(c<START>, c<STOP>) , but I don't really see the needs for it as IMO if the size can be known at compile-time, then you probably better know it than the bounds, especially if you have to think about whether the upper bound is inclusive or exclusive.What do you all think about it?gael
|Mail converted by MHonArc 2.6.19+||http://listengine.tuxfamily.org/|