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

Maybe Several Changes should be made for the newest version of theano? #2

Open
RaymondAmesSpruance opened this issue Apr 14, 2016 · 0 comments

Comments

@RaymondAmesSpruance
Copy link

RaymondAmesSpruance commented Apr 14, 2016

Hello Sir !
I am a student from a university in China and I am trying to reproduce your experimental results on Windows10
with the newest version of theano. I came with several errors and solved them. Maybe there are several changes which should be made
to run BinaryConnect with the newest version of theano ?
[1] As the newest version of theano changed "signal.downsample.max_pool_2d", line 415 in layer.py should be changed to
"z = T.signal.pool.pool_2d(input=z, ds=self.pool_shape, st=self.pool_stride, mode='max')"
...I hope I am right...
[2]I came with a problem between "float32" and "float64" and this finally worked
Chnage line 272 to 276 in trainer.py to:
self.shared_x.set_value(float32(set.X[start:(size+start)]))
self.shared_y.set_value(float32(set.y[start:(size+start)]))
Maybe adding float32 to it can me the program stronger?

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

No branches or pull requests

1 participant