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

Add SameDiff Layer #4426

Closed
wants to merge 26 commits into from
Closed

Add SameDiff Layer #4426

wants to merge 26 commits into from

Conversation

AlexDBlack
Copy link
Contributor

WIP DO NOT MERGE
Still a long way to go here. Needs lots of testing.

Adds SameDiff layer. In summary, to add a SameDiff layer:

  • Users extend BaseSameDiffLayer (i.e., the config class), which has methods to return the graph, parameter names and shapes, etc.
  • Unlike 'vanilla' DL4J/ND4J layers: users will not usually need to implement a Layer implementation class or a parameter initializer (just the config class above)

For now, I'm focusing on the single layer only case (i.e., no building large SameDiff graph across multiple layers). However, I'm keeping this in mind with the API so it can be added later.

@AlexDBlack AlexDBlack force-pushed the ab_samedifflayer branch 2 times, most recently from 87d2a32 to 4a96c3b Compare December 22, 2017 12:16
@AlexDBlack AlexDBlack force-pushed the ab_samedifflayer branch 2 times, most recently from bebb0a3 to db0d7d0 Compare January 8, 2018 04:42
@AlexDBlack
Copy link
Contributor Author

Closing in favor of: #4675

@AlexDBlack AlexDBlack closed this Feb 19, 2018
@agibsonccc agibsonccc deleted the ab_samedifflayer branch April 9, 2018 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant