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

[bugfix] CaffeFunction ignores pad_w #5463

Merged
merged 1 commit into from Oct 11, 2018

Conversation

koreyou
Copy link
Contributor

@koreyou koreyou commented Oct 10, 2018

This fixes #5462 .

@kmaehashi kmaehashi added cat:bug Bug report or fix. to-be-backported Pull request that should be backported. labels Oct 10, 2018
Copy link
Member

@kmaehashi kmaehashi left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!

@kmaehashi
Copy link
Member

Jenkins, test this please.

@kmaehashi kmaehashi added the st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes. label Oct 11, 2018
@kmaehashi kmaehashi self-assigned this Oct 11, 2018
@chainer-ci
Copy link
Member

Jenkins CI test (for commit 6c446a1, target branch master) succeeded!

@kmaehashi kmaehashi merged commit 2c80366 into chainer:master Oct 11, 2018
@kmaehashi kmaehashi added this to the v6.0.0a1 milestone Oct 11, 2018
kmaehashi added a commit to kmaehashi/chainer that referenced this pull request Oct 11, 2018
[bugfix] CaffeFunction ignores pad_w
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat:bug Bug report or fix. st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes. to-be-backported Pull request that should be backported.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CaffeFunction ignores pad_w from Convolution
3 participants