<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Jun 26, 2018 at 12:58 AM Sebastian Berg <<a href="mailto:sebastian@sipsolutions.net">sebastian@sipsolutions.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, 2018-06-26 at 17:30 +1000, Andrew Nelson wrote:<br>
> On Tue, 26 Jun 2018 at 17:12, Eric Wieser <<a href="mailto:wieser.eric%2Bnumpy@gmail.co" target="_blank">wieser.eric+numpy@gmail.co</a><br>
> m> wrote:<br>
> > > I don't think it should be relegated to the "officially<br>
> > discouraged" ghetto of `.legacy_index`<br>
> > <br>
> > The way I read it, the new spelling lof that would be the explicit<br>
> > but not discouraged `image.vindex[rr, cc]`.<br>
> > <br>
> <br>
> If I'm understanding correctly what can be achieved now by `arr[rr,<br>
> cc]` would have to be modified to use `arr.vindex[rr, cc]`, which is<br>
> a very large change in behaviour. I suspect that there a lot of<br>
> situations out there which use `arr[idxs]` where `idxs` can mean one<br>
> of a range of things depending on the code path followed. If any of<br>
> those change, or a mix of nomenclatures are required to access the<br>
> different cases, then havoc will probably ensue.<br>
<br>
Yes, that is true, but I doubt you will find a lot of code path that<br>
need the current indexing as opposed to vindex here,</blockquote><div><br></div><div>That's probably true! But I think it's besides the point. I'd wager that most code paths that will use .vindex would work perfectly well with current indexing, too. Most of the time, people aren't getting into the hairy corners of advanced indexing.</div><div><br></div><div>Adding to the toolbox is great, but I don't see a good reason to take out the ones that are commonly used quite safely.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> and the idea was<br>
to have a method to get the old behaviour indefinitely. You will need<br>
to add the `.vindex`, but that should be the only code change needed,<br>
and it would be easy to find where with errors/warnings.<br></blockquote><div><br></div><div>It's not necessarily hard; it's just churn for no benefit to the downstream code. They didn't get a new feature; they just have to run faster to stay in the same place.</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">Robert Kern</div></div>