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

lastgenre: adding classical music #1240

Merged
merged 4 commits into from Jan 18, 2015
Merged

lastgenre: adding classical music #1240

merged 4 commits into from Jan 18, 2015

Conversation

ghost
Copy link

@ghost ghost commented Jan 18, 2015

Pullrequest for this issue: #1239

Horla added 2 commits January 18, 2015 22:11
Add classic genre with its subgenres
Add classical subgenres
@Freso Freso changed the title Lastegenre: adding classical music lastgenre: adding classical music Jan 18, 2015
@@ -156,6 +156,31 @@
- timba
- twoubadou
- zouk
- classic:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't classical be a better "super genre"? classic sounds a bit odd to me.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably, I am not a native speaker. "classical" (Built by 71,025 people (Used 574,814 times)) seems more popular than "classic" (Built by 26,016 people (Used 115,887 times)) on last.fm.

Changing "classic" to "classical"
- cantata
- chamber music:
- string quartet
- classic
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this should be here at all, TBH. :) This could be classic rock, classic punk, a generic classic, etc. Do people actually use this to tag classical music?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've not imported my classical CD's yet, but when I do it would be great to have a Classical genre, and not classic - as you point out, that means something different and is far too vague.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, as it is now, classic is given as a subgenre of classical, so it might be okay. I just think it's a too vague, non-specific term to be useful as a genre tag.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I see what you mean. I don't think that's the right place for classic, wouldn't you end up having things like classic rock being tagged as classical by mistake then?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, classic rock is listed elsewhere in the genre tree, but the tag is likely to be used for all sorts of things on Last.fm, not just classical music.

Removing "classic"
@Freso Freso merged commit 587ff39 into beetbox:master Jan 18, 2015
Freso added a commit that referenced this pull request Jan 18, 2015
Merge branch 'Horla-master'

From pull request #1240 by @Horla. Fixes #1239.
@Freso
Copy link
Member

Freso commented Jan 18, 2015

Looks good! I've merged it in now with 61797e7 - thank you! :)

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

Successfully merging this pull request may close these issues.

2 participants