search

cerndb / dist-keras

Distributed Deep Learning, with a focus on distributed training, using Keras and Apache Spark.
http://joerihermans.com/work/distributed-keras/
GNU General Public License v3.0
624 stars 169 forks source link

empty byte buffer: use b'' or bytearray() #21

Closed natbusa closed 7 years ago

natbusa commented 7 years ago

'' is a str in python 3 ...

JoeriHermans commented 7 years ago

Yes, sorry, this is an issue I still need to commit (was gone for the weekend).

Change this in networking.py

byte_buffer = ''

to

byte_buffer = b''

I'll do a commit later this evening to fix this.

Edit: You already did a PR, nice. I'll merge it with master! Thanks again.