Skip to content

Make sure to not count the blas kernel init time in the speed tests.#182

Closed
abergeron wants to merge 1 commit intolisa-lab:masterfrom
abergeron:speed
Closed

Make sure to not count the blas kernel init time in the speed tests.#182
abergeron wants to merge 1 commit intolisa-lab:masterfrom
abergeron:speed

Conversation

@abergeron
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread code/test.py
# this time in the runs below.
import pygpu
a = pygpu.empty((2, 2), dtype='float32',
context=theano.gpuarray.type.get_context(None))
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.

Why not do this in theano/gpuarray/init.py use() function?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I guess we could do it, yes.

@abergeron
Copy link
Copy Markdown
Contributor Author

See Theano/Theano#5436 instead.

@abergeron abergeron closed this Jan 20, 2017
@abergeron abergeron deleted the speed branch January 20, 2017 17:59
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