Skip to content
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

Permute Layer #149

Merged
merged 3 commits into from
Aug 3, 2021
Merged

Permute Layer #149

merged 3 commits into from
Aug 3, 2021

Conversation

therealansh
Copy link
Contributor

@therealansh therealansh commented Jun 22, 2021

This PR adds Permute Layer. Resolves #142.

TODO:

  • Add implementation of Permute
  • Export to ModelSaver and ModelLoader
  • Add Docs
  • Add unit tests

Copy link
Contributor

@mkaze mkaze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! I just left some small comments.

@therealansh therealansh changed the title [WIP] Permute Layer Permute Layer Jun 24, 2021
@zaleslaw zaleslaw added the LGTM PR reviewed and is ready to merge label Aug 3, 2021
@zaleslaw
Copy link
Collaborator

zaleslaw commented Aug 3, 2021

@therealansh Please, merge the master branch here

@therealansh
Copy link
Contributor Author

@zaleslaw did this too

@zaleslaw zaleslaw merged commit 53f91b7 into Kotlin:master Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LGTM PR reviewed and is ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Permute layer
3 participants