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.

Sign up for GitHub

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

Jump to bottom

Use chainerx.clipped_relu in F.clipped_relu #7588

Merged
merged 1 commit into from
Jun 25, 2019
Merged

Use chainerx.clipped_relu in F.clipped_relu #7588

merged 1 commit into from
Jun 25, 2019

Conversation

hvy
Copy link
Member

@hvy hvy commented Jun 24, 2019

Following up on #7118.

@hvy hvy added ChainerX Related to ChainerX. cat:performance Performance in terms of speed or memory consumption. labels Jun 24, 2019
@niboshi
Copy link
Member

niboshi commented Jun 25, 2019

Jenkins, test this please

@niboshi niboshi added cat:code-fix Code refactoring that does not change the behavior. and removed cat:performance Performance in terms of speed or memory consumption. labels Jun 25, 2019
@niboshi niboshi added this to the v7.0.0b2 milestone Jun 25, 2019
@niboshi niboshi added the st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes. label Jun 25, 2019
@niboshi niboshi self-assigned this Jun 25, 2019
@niboshi niboshi changed the title Optimize clipped_relu with ChainerX Use chainerx.clipped_relu in F.clipped_relu Jun 25, 2019
@chainer-ci
Copy link
Member

Jenkins CI test (for commit fccbbf4, target branch master) failed with status FAILURE.

@niboshi
Copy link
Member

niboshi commented Jun 25, 2019

LGTM

@niboshi niboshi merged commit e48db81 into chainer:master Jun 25, 2019
@hvy hvy deleted the chx-ch-clipped-relu branch June 25, 2019 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@niboshi niboshi

Labels
cat:code-fix Code refactoring that does not change the behavior. ChainerX Related to ChainerX. st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes.
Projects
None yet
Milestone
v7.0.0b2
Development

Successfully merging this pull request may close these issues.
3 participants
@hvy @niboshi @chainer-ci