ENH: add by_key

[tacaswell]

Promote _transpose to the public method by_key.

[tacaswell]

@efiring @mdboom @jenshnielsen

I think this takes care of the cycler side of what we discussed. We probably should do a new cycler release very soon.

[WeatherGod]

Will this still work even with other operators that I plan to add soon?
http://www.cs.odu.edu/~toida/nerzic/content/set/set_op_prop.html

On Mon, Nov 30, 2015 at 5:27 PM, Thomas A Caswell notifications@github.com
wrote:

@efiring https://github.com/efiring @mdboom https://github.com/mdboom
@jenshnielsen https://github.com/jenshnielsen

I think this takes care of the cycler side of what we discussed. We
probably should do a new cycler release very soon.

—
Reply to this email directly or view it on GitHub
#26 (comment).

[tacaswell]

It should be because under the hood it just does list(self) which exercises all of the operations and gives you back a dict of lists which if you compose with only + is equivalent.

Doing this costs you all of your operator logic, but it works.

[WeatherGod]

yup, I think you are right. For some reason, I had a thought over the vacation that this wouldn't work, but I can't for the life of me remember why I thought that...

[mdboom]

Do we need this, since it was private to begin with?

[tacaswell]

I was 50/50 on this. I was thinking of being semi-evil and using the private _transpose in mpl so that it will work with current tagged version of cycler so we don't have to do a full release cycle on cycler before implementing the key introspection on mpl.

[mdboom]

Ah -- that seems like more than a good enough reason.

[mdboom]

This should probably have a test. Other than that, 👍 from me.

[tacaswell]

bump this should be merged.