Error when training

Question

Error when training

kevaday opened this issue 6 years ago · comments

Whenever I run the training script, even using the original training data, I get the following error:
TypeError: expected 0 arguments, got 1
Here is the full output:
Warning (from warnings module):
File "E:\Python\3.6.2\lib\site-packages\h5py_init_.py", line 36
from .conv import register_converters as register_converters
FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
Using TensorFlow backend.
Traceback (most recent call last):
File "E:\Python\Projects\Classical-Piano-Composer-master\lstm.py", line 121, in
train_network()
File "E:\Python\Projects\Classical-Piano-Composer-master\lstm.py", line 17, in train_network
notes = get_notes()
File "E:\Python\Projects\Classical-Piano-Composer-master\lstm.py", line 37, in get_notes
parts = instrument.partitionByInstrument(midi)
File "E:\Python\3.6.2\lib\site-packages\music21\instrument.py", line 1842, in partitionByInstrument
p.insert(subStream.elementOffset(e), e)
File "E:\Python\3.6.2\lib\site-packages\music21\stream_init.py", line 1701, in insert
self.coreGuardBeforeAddElement(element)
File "E:\Python\3.6.2\lib\site-packages\music21\stream\core.py", line 313, in coreGuardBeforeAddElement
(element, id(element), self, id(self)))
File "E:\Python\3.6.2\lib\site-packages\music21\stream_init.py", line 245, in repr
return super().repr(self)
TypeError: expected 0 arguments, got 1

If anyone could help, I would very much appreciate it,
Thanks

Kevi Aday commented 6 years ago

awesome

Sigurður Skúli Sigurgeirsson · Answer 1 · Sat May 05 2018 22:58:47 GMT+0800 (China Standard Time)

Hi, Yeah I accidentally broke it... I'm sorry. I'm putting together a new computer and I'll fix it as soon as I can. :) Sincerely, Sigurđur Skúli

…

On Sat, 5 May 2018, 14:35 kevaday, ***@***.***> wrote: Whenever I run the training script, even using the original training data, I get the following error: TypeError: expected 0 arguments, got 1 Here is the full output: Warning (from warnings module): File "E:\Python\3.6.2\lib\site-packages\h5py_*init*_.py", line 36 from . *conv import register_converters as register_converters FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type. Using TensorFlow backend. Traceback (most recent call last): File "E:\Python\Projects\Classical-Piano-Composer-master\lstm.py", line 121, in train_network() File "E:\Python\Projects\Classical-Piano-Composer-master\lstm.py", line 17, in train_network notes = get_notes() File "E:\Python\Projects\Classical-Piano-Composer-master\lstm.py", line 37, in get_notes parts = instrument.partitionByInstrument(midi) File "E:\Python\3.6.2\lib\site-packages\music21\instrument.py", line 1842, in partitionByInstrument p.insert(subStream.elementOffset(e), e) File "E:\Python\3.6.2\lib\site-packages\music21\stream_init.py", line 1701, in insert self.coreGuardBeforeAddElement(element) File "E:\Python\3.6.2\lib\site-packages\music21\stream\core.py", line 313, in coreGuardBeforeAddElement (element, id(element), self, id(self))) File "E:\Python\3.6.2\lib\site-packages\music21\stream_init*.py", line 245, in *repr* return super().*repr*(self) TypeError: expected 0 arguments, got 1 If anyone could help, I would very much appreciate it, Thanks — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#4>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFj9p7DOZWgpNOzPCxfkh9qO1Ixz_lbDks5tvbiggaJpZM4TzqN-> .

Kevi Aday · Answer 2 · Sat May 05 2018 23:11:45 GMT+0800 (China Standard Time)

Ok cool! thanks. By the way the this error that I was getting was on Windows. I also tried it on ubuntu and it just crashed. Now I tried it on Kali Linux (which is essentially debian) and it worked!
The music generation was working on Windows but the training wasn't.
Also, I really like your program, it's very cool!
Thanks for the response,
kev

Kevi Aday · Answer 3 · Sun May 06 2018 00:42:13 GMT+0800 (China Standard Time)

actually I thought that the training was working on debian (kali linux) but I still got the original result sadly. I hope you can fix this 👍

PenguinProgrammer · Answer 4 · Sun May 06 2018 15:47:38 GMT+0800 (China Standard Time)

This issue, and potentially a follow up issue can be worked around. I just ran this on windows 10 64 bit.

The error you see is a warning which can be circumvented by the following command
"pip install h5py==2.8.0rc1" (I think a 0, but possibly an upper case O)
The warning is slated to be fixed when the next version of h5py comes out.

The second error you'll run into is if you try to open the wrong file name in predict.py.
This can be fixed by writing "weights.hdf5" in line 71. Although if you rename the weight files, you'll have to change it to match.

This program is really cool, thanks for sharing it!

Kevi Aday · Answer 5 · Sun May 06 2018 22:00:19 GMT+0800 (China Standard Time)

Cool! thanks Penguin. Yea I know about the weights file, I just added a variable at the top so that it can easily be changed.
Thanks for the help man!

Kevi Aday · Answer 6 · Sun May 06 2018 22:46:36 GMT+0800 (China Standard Time)

So I tried doing what you did but I still got the same error.

Sigurður Skúli Sigurgeirsson · Answer 7 · Mon May 07 2018 03:41:29 GMT+0800 (China Standard Time)

I have pushed a fix to the bug. The problem was that I had added some MIDI files that did not contain parts to the midi_songs directory but I never finished rewriting the parser in the master branch. It should work now.

…

On Sun, May 6, 2018 at 2:46 PM, kevaday ***@***.***> wrote: So I tried doing what you did but I still got the same error. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#4 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFj9p6PHWN91ODHGa-7r5tg9NyB20HLAks5tvwzMgaJpZM4TzqN-> .

Kevi Aday · Answer 8 · Mon May 07 2018 05:22:15 GMT+0800 (China Standard Time)

Awesome, I'll try it!

Kevi Aday · Answer 9 · Mon May 07 2018 05:22:29 GMT+0800 (China Standard Time)

this should work on windows, right?

Sigurður Skúli Sigurgeirsson · Answer 10 · Mon May 07 2018 05:23:34 GMT+0800 (China Standard Time)

Yes, I haven't set up Linux yet on my new computer so I tested the fix on Windows. :)

…

On Sun, May 6, 2018 at 9:22 PM, kevaday ***@***.***> wrote: this should work on windows, right? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#4 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFj9p-ewssJV49NwKjqMeKmNs5ScTm8Oks5tv2mVgaJpZM4TzqN-> .

Kevi Aday · Answer 11 · Mon May 07 2018 05:35:34 GMT+0800 (China Standard Time)

so you updated the repository, right?

Sigurður Skúli Sigurgeirsson · Answer 12 · Mon May 07 2018 05:38:56 GMT+0800 (China Standard Time)

hahah, sorry I only committed them, I'm pushing the update now.

…

On Sun, May 6, 2018 at 9:35 PM, kevaday ***@***.***> wrote: so you updated the repository, right? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#4 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFj9p7FzPO6z_TckHUSjSkKBMdd1qOUTks5tv2yngaJpZM4TzqN-> .

Kevi Aday · Answer 13 · Mon May 07 2018 05:41:13 GMT+0800 (China Standard Time)

Phew, I got scared there a second because I got the same error lol