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.

Sign up for GitHub

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

Jump to bottom

PermutedDimsArray #576

Merged

MikeInnes merged 2 commits into FluxML:master from mcabbott:patch-1

Jan 29, 2019

Member

mcabbott commented Jan 28, 2019 •

edited

Loading

This copies the gradient of permutedims to also work on the lazy version PermutedDimsArray.

Travis CI says ERROR: Unsatisfiable requirements detected for package DiffRules which surely this could not cause?

mcabbott added 2 commits

January 28, 2019 18:15


          PermutedDimsArray like permutedims

031d1b3

e.g. PermutedDimsArray(rand(2,3) |> param, (2,1))


          PermutedDimsArray test

55a7359

Member

MikeInnes commented Jan 29, 2019

Yes, that was a separate CI issue. Thanks!

MikeInnes merged commit 0469394 into FluxML:master

mcabbott deleted the patch-1 branch

July 24, 2019 12:17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet