kellnerd / harmony

Music Metadata Aggregator and MusicBrainz Importer
MIT License
55 stars 7 forks source link

Adding artist urls doesn't give all the URLs that Harmony used during release import #71

Open RustyNova016 opened 1 week ago

RustyNova016 commented 1 week ago

Sometimes when importing releases, when I add the URLs of the matched artists, not all the websites that were used during release importing are added to the artist

For exemple, I added NightOwl by Viperactive and RISK from spotify. I had to create RISK as this artist wasn't in the database yet. But when adding the links of the artist, only deezer was added by harmony.

Harmony link: https://harmony.pulsewidth.org.uk/release?spotify=1IqahQZWVdxO8ZMAsbO0hE&gtin=723277178578&deezer=&itunes=&tidal=&region=GB&ts=1731955338

Release edit: https://musicbrainz.org/edit/118110916

Artist edit: https://musicbrainz.org/edit/118110937

kellnerd commented 1 week ago

Thank you for the example. The reason why not all artist URLs are seeded here is that the different providers return different artist credits (Spotify returns two artists, Deezer returns three). Improving the overly simplistic artist matching algorithm (#54) should solve this issue.

RustyNova016 commented 1 week ago

Well as long as there's enough links to link an existing artist without searching through the 69137 other "risk"s it's fine.