AttributeError: 'collections.OrderedDict' object has no attribute 'iteritems' about neural-storyteller

[HappyCoderMan]

Get the Python3 compatible version of skipthoughts.py here:
https://github.com/ryankiros/skip-thoughts/pull/53/files

I'm also on Anacnoda x64, and my path_to_skipthoughts looks like this:
path_to_skipthoughts = 'C:\neuralstoryteller\skip-thoughts-master\'
I have the models in a subfolder of that path named "models", and this works.

You can also just change iteritems() to items() in most of these cases to fix that error.

[HappyCoderMan]

Please check your bi_skip.npz file. It may be corrupt or missing if the error says it's not a valid zip file.
My code makes that file name like this: path_to_bmodel = path_to_models + 'bi_skip.npz'
Mine has this size: 289,340,074 bytes
Mine is in this folder: ./models/
I have a bi_skip.npz.pkl file at the same path that is 689 bytes.

The config.py sets the relative path like this:

Skip-thoughts

paths['skmodels'] = './models/'
paths['sktables'] = './models/'

[SundareshPrasanna]

Hi @ErikOmberg thanks for the Python3 compatible version.. when i try to use the new skip-thought code (py3 compatible one), i get an error message as below:

C:\Users\Dell\Documents\Neural_storyteller\neural_storyteller\romance.npz
Loading skip-thoughts...
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Dell\Documents\Neural_storyteller\neural-storyteller-master\generate.py", line 86, in load_all
config.paths['sktables'])
TypeError: load_model() takes 0 positional arguments but 2 were given

Can u help me with this? I see that we are passing 2 arguments config.paths['skmodels'], config.paths['sktables'] in the original generate.py.. while in the new skipthought i see load_model does not take any arguments.. so i tried to change it to just
stv = skipthoughts.load_model()
and got the error:

Loading model parameters...
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Dell\Documents\Neural_storyteller\neural-storyteller-master\generate.py", line 85, in load_all
stv = skipthoughts.load_model()
File "C:\Users\Dell\Documents\Neural_storyteller\neural-storyteller-master\skipthoughts.py", line 55, in load_model
bparams = load_params(path_to_bmodel, bparams)
File "C:\Users\Dell\Documents\Neural_storyteller\neural-storyteller-master\skipthoughts.py", line 259, in load_params
pp = numpy.load(path)
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\site-packages\numpy\lib\npyio.py", line 412, in load
pickle_kwargs=pickle_kwargs)
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\site-packages\numpy\lib\npyio.py", line 171, in init
_zip = zipfile_factory(fid)
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\site-packages\numpy\lib\npyio.py", line 101, in zipfile_factory
return zipfile.ZipFile(file, *args, **kwargs)
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\zipfile.py", line 1026, in init
self._RealGetContents()
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\zipfile.py", line 1093, in _RealGetContents
raise BadZipFile("File is not a zip file")
zipfile.BadZipFile: File is not a zip file

[HappyCoderMan]

You're right - I see that link I posted only had a loadmodel function with no arguments. I didn't use that code directly. I started with the master code, and made some code changes (based on that link) to get my py3 working.

The necessary changes aren't too bad. It's mostly a few "items" change to "iteritems", parenthesis around print statements, etc...

[SundareshPrasanna]

I agree.. but i am getting an error when i do so :( could u help me with the error below

Loading skip-thoughts...
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Dell\Documents\Neural_storyteller\neural-storyteller-master\generate.py", line 86, in load_all
config.paths['sktables'])
File "C:\Users\Dell\Documents\Neural_storyteller\neural-storyteller-master\skipthoughts.py", line 39, in load_model
bparams = load_params(path_to_bmodel, bparams)
File "C:\Users\Dell\Documents\Neural_storyteller\neural-storyteller-master\skipthoughts.py", line 174, in load_params
pp = numpy.load(path)
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\site-packages\numpy\lib\npyio.py", line 412, in load
pickle_kwargs=pickle_kwargs)
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\site-packages\numpy\lib\npyio.py", line 171, in init
_zip = zipfile_factory(fid)
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\site-packages\numpy\lib\npyio.py", line 101, in zipfile_factory
return zipfile.ZipFile(file, *args, **kwargs)
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\zipfile.py", line 1026, in init
self._RealGetContents()
File "C:\Users\Dell\Anaconda3\envs\tensorflow-gpu\lib\zipfile.py", line 1093, in _RealGetContents
raise BadZipFile("File is not a zip file")
zipfile.BadZipFile: File is not a zip file

[SundareshPrasanna]

Thanks a lot @ErikOmberg .. i didn't notice that my file was corrupted.. i have one question tho wrt resize in generate.py

line 154: im = skimage.transform.resize(im, (256, w*256/h), preserve_range=True)
here, there is a possibility that the number won't end up as integer.. so should we add int() to round it up?

[HappyCoderMan]

Yes, I made that change too:
int(w*256/h)

[SundareshPrasanna]

Thanks @ErikOmberg .. I've raised a new issue #37 i was wondering if u could help me with that or have u faced similar issues while running your codes.