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

duplicate albums #43

Open
gilcoh33 opened this issue Aug 31, 2014 · 22 comments
Open

duplicate albums #43

gilcoh33 opened this issue Aug 31, 2014 · 22 comments

Comments

@gilcoh33
Copy link

instead of showing the album, I think it shows the album artist instead.

didn't happen in previous releases - 0.x

@foreverguest
Copy link
Collaborator

which version are you using?

@gilcoh33
Copy link
Author

1.1

@gilcoh33
Copy link
Author

gilcoh33 commented Nov 4, 2014

Hi,

any news on this issue?
im on exp-1.1.1 still duplicated albums appearence

any dev version can solve it?

10x

@foreverguest
Copy link
Collaborator

Hi,

I falied to catch this problem in my library, so I'm not sure what could cause this problem for you.

@gilcoh33
Copy link
Author

gilcoh33 commented Nov 4, 2014

Do you need some logs?
I retested again the 0.8.0 version and this is Ok in this case.

@foreverguest
Copy link
Collaborator

Could you send a print screen showing the problem?

@gilcoh33
Copy link
Author

gilcoh33 commented Nov 6, 2014

I think I know what the problem is, in previous versions the Album list was distributed with the "Albums" tagging, in this version (1.x) the Album is distributed with "Album artist" tagging so thats why I see duplicated albums instead of the album name itself, can you approve this?
img_20141107_005248

This must be changed, there are many people that listen to albums that contain various of artists, please change the code to be listed as Album, NOT album artist.

thanks

@foreverguest
Copy link
Collaborator

Actually I have changed artist -> album artist in some places.
Could you use a view where we can see the albums names?

@gilcoh33
Copy link
Author

gilcoh33 commented Nov 7, 2014

So I think you should change also album_artist - > album.

See in the picture that the same album is seperated cause he has couple of artist in it.

@foreverguest
Copy link
Collaborator

So you are browsing 'My library' - > 'Artists'?

@gilcoh33
Copy link
Author

gilcoh33 commented Nov 7, 2014

No my friend, I'm browsing Albums.

Anyway I don't think you should use the "album artist" tag at all.

Even the original Google Music web app not using it.

Artist is Artist tagging
Album is Album tagging

@gilcoh33
Copy link
Author

Found this on the API code:

if albumArtist:
query = "select * from songs where album = :filter and album_artist = :albumArtist order by disc asc, track asc"
elif filter_type == 'album':

as I thought it looks like the code filter "Albums" as "album_artist".
this should be replaced with "album" since there are some albums with different artists --> 1 album with 10 different artists (like for Top40, or some season collections)

Thanks

@foreverguest
Copy link
Collaborator

It is my understanding that the 'album_artist' should be the same for all tracks from a single album.
It is the 'artist' tag that varies for each track.
Usually on albums with several artists the 'album_artist' has something like 'Various Artists'.

@gilcoh33
Copy link
Author

I got many official albums and many of them not using the "Various Artists", and if they are it will be the same result when many songs with "Various Artists" will be collected to the same album, and it will be huge.

in version 0.8.x the album was filtered for 'album', as I think it should be:

if filter_type == 'album':
query = "select * from songs where album = ? order by disc asc, track asc"
elif filter_type == 'artist':
query = "select * from songs where artist = ? order by album asc, disc asc, track asc"
else:
query = "select * from songs where genre = ? order by title asc"

@foreverguest
Copy link
Collaborator

Although I disagree about the use of the 'album_artist' tag, I agree that we should find a way to prevent the same album from being listed more than once.
The problem is that albums from different artists may have the same name, I thought that distinguishing each album by 'album'+'album_artist' was the best way to prevent any confusion.
Do you have other suggestion to deal with same name albums?

@gilcoh33
Copy link
Author

You're doing a wonderful job, and I guess you'll know how to handle it better then me,
I just want to get rid of the duplications.

It just weird that this issue wasn't appear on previous versions.
I hope you'll find a way soon

thanks again.

@gilcoh33
Copy link
Author

Hi
Thanks for the update - I want to test it, is it already applied? version 0.8.2?

@rui-castro
Copy link
Contributor

Hi @gilcoh33 ,

It's not applied yet.
If you want to try my version, install my repo (http://kodi.ruicastro.me/repository.ruicastro/repository.ruicastro-1.0.1.zip) and install or update the plugin Google Music.

@gilcoh33
Copy link
Author

Oh good.
I'll try it

Thanks man, appreciate it

@gilcoh33
Copy link
Author

Hi @rui-castro

Your latest version is 1.2 right?
Is it patched with this fix? cause I still see album as album artist

@rui-castro
Copy link
Contributor

Hi @gilcoh33 ,

My version is '1.0.0_ruicastro_0.8.2'.
Here's a direct link to the plugin http://kodi.ruicastro.me/plugin.audio.googlemusic/plugin.audio.googlemusic-1.0.0_ruicastro_0.8.2.zip.

@gilcoh33
Copy link
Author

Thanks a lot @rui-castro !!! looks amazing - you just made my day

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

3 participants