facebookarchive / tutorials

Caffe2 Tutorials
Apache License 2.0
124 stars 60 forks source link

Let's make sure protobuf files are loaded with "rb" #1

Closed orionr closed 6 years ago

orionr commented 6 years ago

People have had some issues on Windows using the tutorials

For example, https://github.com/caffe2/models/issues/9

We want to change

with open(<protobuf filename>) as f:

to

with open(<protobuf filename>, 'rb') as f:

in all cases where we are reading in the protobuf files.