Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Impl benchmarks for spatial pooling #17

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Impl benchmarks for spatial pooling #17

wants to merge 10 commits into from

Conversation

asi1024
Copy link
Member

@asi1024 asi1024 commented Apr 12, 2018

No description provided.

@kmaehashi
Copy link
Member

kmaehashi commented Jul 4, 2018 •
edited
Loading

Sorry for leaving this PR for months.

  • I feel some data used in these benchmarks are too small to detect regression. (sorry for making a qualitative comment...) For example, AveragePooling2D and MaxPooling2D runs in less than 1ms even with CPU, and GPU time is slower than CPU.
  • Some benchmarks need fix to work with the latest Chainer. (e.g., number of arguments changed in spatial_pyramid_pooling_2d)

@kmaehashi kmaehashi self-assigned this Jul 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@kmaehashi kmaehashi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@asi1024 @kmaehashi