Batcher: Prevent integer overflow when calculating batchStart

hughperkins / DeepCL

OpenCL library to train deep convolutional neural networks

Mozilla Public License 2.0

866 stars 200 forks source link

Closed merceyz closed 7 years ago

merceyz commented 7 years ago

After adding some more training data DeepCL started crashing during training, turns out batch * batchSize caused an integer overflow.

This PR fixes that by using a 64bit int instead of a 32bit one

merceyz commented 7 years ago

@hughperkins Anything preventing this from getting merged?

hughperkins commented 7 years ago

I didnt notice it basically :-) Thanks! :-)