search

uoguelph-mlrg / theano_alexnet

Theano-based Alexnet
BSD 3-Clause "New" or "Revised" License
229 stars 113 forks source link

Minor error should be theano instead of T #5

Closed qyouurcs closed 9 years ago

qyouurcs commented 9 years ago

This could not even compile.

gwding commented 9 years ago

thanks for pointing this out! it seems that it should theano.shared Just curious which version of theano are you using? as I never encountered compilation issue because of this. and I've tested on version 0.7.0rc1.dev-ca45a0bc561e68f9e6ec93a9541e062b7df10802 T.shared and tensor.shared seem to return the same thing. Although T.shared is deprecated.

qyouurcs commented 9 years ago

On 2015-03-26 17:30, Weiguang (Gavin) Ding wrote:

thanks for pointing this out! it seems that it should theano.shared Just curious which version of theano are you using, as I never encountered compilation issue because of this. and I've tested on version 0.7.0rc1.dev-ca45a0bc561e68f9e6ec93a9541e062b7df10802 T.shared and tensor.shared seem to return the same thing. Although T.shared is deprecated.

Reply to this email directly or view it on GitHub [1].

*

Links:

[1] https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_uoguelph-2Dmlrg_theano-5Falexnet_pull_5-23issuecomment-2D86726577&d=AwMCaQ&c=kbmfwr1Yojg42sGEpaQh5ofMHBeTl9EI2eaqQZhHbOU&r=5zmTjP9cXtM3_VzL4Px9ohU2o4QfFGE-x2iPcnNNBzY&m=vELVasJStmukdagoNpxYCc_THBhqI7mvUqW4G-7uses&s=t4Owm_uszicfS_haV4OFy6HG5adDSk7CAPJt5Ay7m7o&e=

I am using the latest version (bloody version). I checked out the Theano code from github directly. It should be 0.7rc2.

Quanzeng