-
Notifications
You must be signed in to change notification settings - Fork 498
Music Genre Classification #702
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few Changes:
- Remove the Database
- Upload the Model File on G-Drive/Dropbox and drop the link here
- Make changes on Python README as well
|
|
||
| ## Dataset | ||
|
|
||
| We have used GTZAN dataset for our task. It is publicly available, here is the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since you have already referenced the GTZAN Dataset, do we actually need to store it in the repo as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't get that, I have just referenced the Repo here, and have stored a file containing 6 features, the whole DB is 1.2 gb.
Can you clarify this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The size of the csv is also half MB, if you want I can get rid of the file and paste a drive link there
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah. The only intent was to get rid of the files that can be retrieved from online sources.
|
@HarshCasper Would it be fine if I update |
Fixed in #720 |
|
Thanks for the PR! I am inclined towards merging the same, but can you take a look at the failing checks and the reasons they are failing. As far as I can notice, the Spell Check Action is generating a lot of unnecessary noise. The LinkedIn URL also directs to 999 Status Code, which is understandable since LinkedIn filter requests based on the user-agent. Can you also try looking at the other errors and possibly fix them or raise an Issue to be fixed in these actions? |
|
Can you check #672 @HarshCasper |
Description
Deep Learning Implementation of Music Genre Classification.
Fixes #576
Type of change
Checklist:
README.md