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

Lower media weight if catalognum exists #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MxMarx
Copy link

@MxMarx MxMarx commented Jun 12, 2023

Beets doesn't seem to penalized the match distance if a release is missing a catalog number (as far as I can tell but I could easily be wrong).

If, for example, a release has a CD and digital version, and the digital one is missing the catalog number, then if media is removed from the query because the origin file has a catalog number, both media and catalog number might end up being ignored.

The release distance isn't lowered if it doesn't have a catalog number (as far as I can tell).
If the media is removed, then if one release matches both the media and catalog number, and another has a non-matching media and doesn't have a catalog number, then the non-matching media wouldn't be penalized.

For examples, this plugin chooses the cassette release for this album instead of the CD, even though both media and catalog number match in origin.yaml
https://musicbrainz.org/release-group/05c73df7-423f-4dab-b600-afdc7d453e4e
Instead, it might be good to lower the media weight.
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.

1 participant