Skip to content
This repository was archived by the owner on Jul 1, 2023. It is now read-only.

Change operations a bit + add TransposedConv2D layer #64

Merged
merged 9 commits into from
Mar 29, 2019
Merged

Change operations a bit + add TransposedConv2D layer #64

merged 9 commits into from
Mar 29, 2019

Conversation

tanmayb123
Copy link
Contributor

@tanmayb123 tanmayb123 commented Mar 29, 2019 •
edited
Loading

Closes #53

@tanmayb123
Copy link
Contributor Author

I've tested this out locally, and everything seems to be working. @rxwei - do you have some feedback on how this implementation could be made better?

@tanmayb123 tanmayb123 mentioned this pull request Mar 29, 2019
Closed
rxwei
rxwei reviewed Mar 29, 2019
View reviewed changes
rxwei
rxwei reviewed Mar 29, 2019
View reviewed changes
@tanmayb123
Copy link
Contributor Author

I believe I've fixed all the above items from your review.

rxwei
rxwei approved these changes Mar 29, 2019
View reviewed changes
@rxwei
Copy link
Contributor

rxwei commented Mar 29, 2019 •
edited
Loading

Could you fix the build failure?

@tanmayb123
Copy link
Contributor Author

It worked.

@rxwei
Copy link
Contributor

rxwei commented Mar 29, 2019

Thanks!

@rxwei rxwei merged commit ee6172c into tensorflow:master Mar 29, 2019
@jcatanza jcatanza mentioned this pull request Apr 23, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rxwei rxwei rxwei approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tanmayb123 @rxwei