epoupon / lms

Lightweight Music Server. Access your self-hosted music using a web interface.
http://lms-demo.poupon.dev
GNU General Public License v3.0
1.14k stars 62 forks source link

, and ; #518

Closed VlaK0r closed 2 months ago

VlaK0r commented 2 months ago

Hello! Thank you for your work!

изображение изображение

There is the following list of artists filled with MusicBrainz Picard.

изображение

Artists are divided ; are counted as one artist. We are talking about [Album artists] изображение

Please tell me how to fix this?

epoupon commented 2 months ago

Ah, I recommend to use a custom tag for that, see https://github.com/epoupon/lms?tab=readme-ov-file#multiple-album-artists

VlaK0r commented 2 months ago

изображение

Thank you! So this setting can be removed? Won't it help?

epoupon commented 2 months ago

Well it is another mean to split, less reliable, but it should have worked too. Did you rescan after having set these separators?

VlaK0r commented 2 months ago

Did you rescan after having set these separators?

Yes, I did, but it did not give a result.

epoupon commented 2 months ago

Ah now I understand, LMS only uses the first delimiter hit. Here the first is , so it considers this is the only one to use for the whole string, to avoid unwanted further splits. But this is highly debatable...

epoupon commented 2 months ago

So not sure to do with this. We could blindly apply all delimiters set, or you could add albumartists to the tags of this album. This makes also me think LMS tries to split tags in artists and albumartists, which may lead to bad results for correctly split artists in multi-valued tags

VlaK0r commented 2 months ago

Ah, I recommend to use a custom tag for that, see https://github.com/epoupon/lms?tab=readme-ov-file#multiple-album-artists

It's okay, your way works. I took advantage of it and everything worked out.

epoupon commented 2 months ago

In order to be more consistent, lms now applies all separators on tags.