Skip to content
This repository has been archived by the owner on Jun 21, 2022. It is now read-only.

JaggedArray._broadcast should accept scalars and any AwkwardArray #94

Closed
jpivarski opened this issue Mar 8, 2019 · 2 comments
Closed

Comments

@jpivarski
Copy link
Member

Two missing cases. Scalars have an obvious implementation. General AwkwardArrays would turn into JaggedArray(IndexArray(...)).

@jpivarski
Copy link
Member Author

Fixed by #99. Actually, we don't have to introduce IndexedArray; we can use the fact that every awkward array has a fancy indexing implementation to apply the rearrangement eagerly.

Really, the only use of IndexedArray is for lazy rearrangements, the only use of which I can think of is cyclic references.

@jpivarski
Copy link
Member Author

Oh, and dictionary encoding. In case you don't want to eagerly create a huge number of strings from indexes, but only want to lazily describe it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant