FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix zero padding in convolve2NN by syurkevi · Pull Request #2820 · arrayfire/arrayfire · GitHub

Repository navigation

fix zero padding in convolve2NN - #2820

Merged
9prady9 merged 2 commits into
arrayfire:masterfrom
syurkevi:issue2817
Apr 8, 2020
Merged

9prady9 merged 2 commits into
arrayfire:masterfrom
syurkevi:issue2817

Conversation

syurkevi commented Apr 2, 2020

Copy link
Copy Markdown
Contributor

zero padding issue due to "empty" dim4 in c++ api.
adds fixes and simple tests addressing #2817

9prady9 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Looks good. Can you please fix the formatting.

9prady9 merged commit decde4e into arrayfire:master Apr 8, 2020
9prady9 deleted the issue2817 branch April 8, 2020 05:07
9prady9 mentioned this pull request Apr 8, 2020
9prady9 added this to the 3.7.2 milestone Apr 8, 2020
umar456 mentioned this pull request Jun 27, 2020
2 tasks done
umar456 pushed a commit to umar456/arrayfire that referenced this pull request Jul 3, 2020
* add tests for zero padding

* fix clang formatting
9prady9 pushed a commit that referenced this pull request Jul 5, 2020
* add tests for zero padding

* fix clang formatting
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL