Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong in Keras_train #5

Open
Jangelaw opened this issue Apr 23, 2018 · 14 comments
Open

Wrong in Keras_train #5

Jangelaw opened this issue Apr 23, 2018 · 14 comments

Comments

@Jangelaw
Copy link

In Keras_train.py line 192: DataGen (), and preprocess.py line 39.
There is nothing in the self.mmdirs. Just wondering what is suppose to be in the mmdir?

@jsleep
Copy link
Owner

jsleep commented Apr 23, 2018 via email

@Jangelaw
Copy link
Author

I've already run the preprocess, but there is nothing is the mmdir.
I think maybe your code is suppose to create the input.dat in train, val and test file?
Also, there is some place which is not workable, you may double check the code of preprocess and improve it in the future.

@Jangelaw
Copy link
Author

BTW, what's the corrent dtype in '.dat' file? uint8 or float?

@jsleep
Copy link
Owner

jsleep commented Apr 23, 2018 via email

@Jangelaw
Copy link
Author

I think you've already made a good start.
I've done some search online and I didn't see many useful deep-learning-based code.
So, if you can improve it, it will very helpful for other researchers to do some work on it.
Recently,I'm preparing a paper on AMT and I need to implement the code of the work 'An End-to-End Neural Network for Polyphonic Piano Music Transcription'. Your code is the only one I can find, which is close to this work. That's why I always work on your code :P
BTW, if you have any AMT-related publication, I'd like to cite your work as well :)

@jsleep
Copy link
Owner

jsleep commented Apr 24, 2018 via email

@Jangelaw
Copy link
Author

Jangelaw commented May 16, 2018 via email

@jsleep
Copy link
Owner

jsleep commented May 16, 2018 via email

@Jangelaw
Copy link
Author

Jangelaw commented May 21, 2018 via email

@justhalf
Copy link

Yes honestly this repo is not in any state right now for public use, I will try to clean this up soon because I’ve been getting a lot of interested people contacting me recently
On Mon, Apr 23, 2018 at 11:07 AM Jangelaw @.***> wrote: BTW, what's the corrent dtype in '.dat' file? uint8 or float? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#5 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AGPOij48XwW_O9mmejPDktANGazlcOSMks5trhhngaJpZM4TgPTk .
-- Jonathan Sleep

Is it happening soon? I'm looking at the code and it seems the code is not runnable (missing variables) as is, I needed to make a few adjustments (updating args with config, for example).

@whartley94
Copy link

Any chance we can get an update on this? Think you've made a great start but would be great if it were more user-friendly! :)

@jsleep
Copy link
Owner

jsleep commented Feb 5, 2019

No update unfortunately, been busy at my full time job and haven't had the drive to program in my off hours. However, I'm currently in the process of switching jobs and if I get a decent break I will for sure be working on this and making a consumable project. I've had fixing this project up in the back of my head for so long now...

@czonios
Copy link

czonios commented Mar 2, 2019

Hey,

I have gotten this project to work, however I used Python 3 and refactored the code completely in the process of fixing and understanding it, using a very different coding style so I'm unsure if I should even do a pull request with these radical changes.
Even if I were to, I've switched away from this project (because I'm implementing something similar in PyTorch instead of Keras, using a different dataset file format, different spectrogram type, different model etc.)

I would be glad, when I get home, to take a look at my notes and give you some info on what changes I made in order for it to work.
I could also clone and push all my changes to my own repository, and then @jsleep could decide whether he wants to merge it or not - the caveat here is I can't put in the time required to test if it's working besides the bare minimum. I can't make any promises about this since I'll need a considerable amount of time to go through it again before I can comfortably push it.

Let me know what you'd prefer and if enough people are interested and @jsleep agrees, I'll see what I can do.

Christos

@jseales
Copy link

jseales commented Jun 26, 2020

Hi czonios, I'm working on a similar project. I'd be interested in seeing your reworked code. Let me know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants