-
-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add PermuteDims
transformation
#89
Conversation
Why not use |
Good point, addressed in 3e44d1d.
Wouldn't this be considered breaking? We could introduce more explicit |
True, but do we really need the explicit versions if we have I think it's more friendly to not have the user worrying |
Sounds good. I will open a separate PR for changes to |
Sounds good, thanks! |
Adds
PermuteDims
, a thin wrapper aroundpermutedims
for use onArrayItem
s.This should help with Flux compatibility (#56):
HWC
toWHC
conversions can be implemented by addingPermuteDims(2, 1, 3)
to a pre-processing pipeline.