Re: [eigen] New indexing/slicing API: almost ready to be merged |
[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]
yes, I've initially thought about head/tail to match the current API. It does read well for vectors, but for matrices it's borderline: A(head(n),tail(m)) and when used alone head(n) to say [0,1,...,n-1] it's really weird.
The capitalized 'N' also help to see the connection with seqN.
If anyone have other better suggestions to offer, I'm all for it!!
Also, for what it's worth, I do prefer 'seqn' to 'seqN'. I feel the word "seq" is already short enough that adding an extra letter differentiates it enough. Of course I do realize that 'N' helps emphasizing the difference between seq and seqN so it helps with maintaining correctness.true story: yesterday I wasted 5mn in debugging a unit test whereas I simply made a typo in writing seq(...) instead of seqN(...), so I would rather re-enforce the differences!
So I propose to:1) Eigen::placeholders::all --> Eigen::All2) move last/end to Eigen::Symbols
Mail converted by MHonArc 2.6.19+ | http://listengine.tuxfamily.org/ |