starred/zotify

Fork 0

mirror of https://github.com/Googolplexed0/zotify.git synced 2026-04-25 06:15:55 +03:00

[GH-ISSUE #23] Filter album types #21

New issue

Closed

opened 2026-02-27 04:57:08 +03:00 by kerem · 10 comments

kerem commented

2026-02-27 04:57:08 +03:00

Owner

Originally created by @U2EZNeko on GitHub (Jun 13, 2025).
Original GitHub issue: https://github.com/Googolplexed0/zotify/issues/23

Originally assigned to: @Googolplexed0 on GitHub.

When i grab a whole discography, especially of older artists, I end up with like 30 Live albums or compilations.

Is it possible to filter out specific album types?

Originally created by @U2EZNeko on GitHub (Jun 13, 2025). Original GitHub issue: https://github.com/Googolplexed0/zotify/issues/23 Originally assigned to: @Googolplexed0 on GitHub. When i grab a whole discography, especially of older artists, I end up with like 30 Live albums or compilations. Is it possible to filter out specific album types?

kerem

2026-02-27 04:57:08 +03:00

closed this issue
added the
enhancement
label

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@Googolplexed0 commented on GitHub (Jun 15, 2025):

I will look into this, could be a nice config flag. It will mostly depend on if the API stores some kind of LIVE album flag like they do for COMPILATION albums. Not really comfortable skipping albums using a song title regex due to very easy false positives no matter what/how a specific filter is implemented.

@Googolplexed0 commented on GitHub (Jun 15, 2025): I will look into this, could be a nice config flag. It will mostly depend on if the API stores some kind of LIVE album flag like they do for COMPILATION albums. Not really comfortable skipping albums using a song title regex due to _very_ easy false positives no matter what/how a specific filter is implemented.

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@U2EZNeko commented on GitHub (Jun 15, 2025):

Very much appreciated. Even skipping compilations would avoid so many duplicate tracks.

Maybe a config option for user-filters?
I wouldn't mind the occasional missing album if I can set Live at*, Live from* or something similar would be neat.

@U2EZNeko commented on GitHub (Jun 15, 2025): Very much appreciated. Even skipping compilations would avoid so many duplicate tracks. Maybe a config option for user-filters? I wouldn't mind the occasional missing album if I can set Live at*, Live from* or something similar would be neat.

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@Googolplexed0 commented on GitHub (Jun 16, 2025):

NO_COMPILATION_ALBUMS config option added with 0afa82785f, not sure how frequently the API actually uses the "album_type" = "compilation" flag though.

User regex filters is something that I actually think would be a really good idea, though it would be very powerful. I'll experiment with that now.

@Googolplexed0 commented on GitHub (Jun 16, 2025): NO_COMPILATION_ALBUMS config option added with 0afa82785fbb6de9210a190b5d26e55c1b7fc6d4, not sure how frequently the API actually uses the `"album_type" = "compilation"` flag though. User regex filters is something that I actually think would be a really good idea, though it would be very powerful. I'll experiment with that now.

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@Googolplexed0 commented on GitHub (Jun 16, 2025):

@U2EZNeko, please check out the new branch user-regex-filters (via 3878a4e). User Regex filters are only implemented for albums right now, but it should be very easy to expand to all other items. I also included a simple example Regex for filtering out live performances in the README in the Example Regex Values section.

EDIT: Expanded functionality to tracks and podcast episodes, don't see much value in adding it for playlists or artists.

Give the branch a beta test and let me know if you find any issues.

@Googolplexed0 commented on GitHub (Jun 16, 2025): @U2EZNeko, please check out the new branch [user-regex-filters](https://github.com/Googolplexed0/zotify/tree/user-regex-filters) (via 3878a4e). User Regex filters are only implemented for albums right now, but it should be very easy to expand to all other items. I also included a simple example Regex for filtering out live performances in the `README` in the `Example Regex Values` section. EDIT: Expanded functionality to tracks and podcast episodes, don't see much value in adding it for playlists or artists. Give the branch a beta test and let me know if you find any issues.

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@U2EZNeko commented on GitHub (Jun 16, 2025):

I'd love to but im running into some rather noob-ish issues.

pipx install git+https://github.com/Googolplexed0/zotify.git@user-regex-filters

doesnt seem to work, how exactly do I install the specific branch with pipx if I may ask?

@U2EZNeko commented on GitHub (Jun 16, 2025): I'd love to but im running into some rather noob-ish issues. pipx install git+https://github.com/Googolplexed0/zotify.git@user-regex-filters doesnt seem to work, how exactly do I install the specific branch with pipx if I may ask?

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@Googolplexed0 commented on GitHub (Jun 16, 2025):

Not your fault, I used an invalid version number (it had letters). Fixed with ea3f62adc795aa269094fa121a64de4215715cf6, the pipx install should work now.

@Googolplexed0 commented on GitHub (Jun 16, 2025): Not your fault, I used an invalid version number (it had letters). Fixed with ea3f62adc795aa269094fa121a64de4215715cf6, the pipx install should work now.

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@U2EZNeko commented on GitHub (Jun 17, 2025):

Pull worked this time.
Using your regex example for now.

I've got my script running, we'll see if it worked tomorrow when the downloads are finished.

On a side note because i couldn't find it in the readme.
Can I set multiple patterns? Whats the delimiter for the config file?

@U2EZNeko commented on GitHub (Jun 17, 2025): Pull worked this time. Using your regex example for now. I've got my script running, we'll see if it worked tomorrow when the downloads are finished. On a side note because i couldn't find it in the readme. Can I set multiple patterns? Whats the delimiter for the config file?

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@Googolplexed0 commented on GitHub (Jun 17, 2025):

There is no explicit delimiter for multiple regexes, however, you can add multiple patterns into a single regex by chaining the "or" construction (|) : (:?<first pattern here>)|(:?<second pattern here>)|(:?<third pattern here>). It can also be recursively stacked, as seen in the Live Performances example.

@Googolplexed0 commented on GitHub (Jun 17, 2025): There is no explicit delimiter for multiple regexes, however, you can add multiple patterns into a single regex by chaining the "or" construction (`|`) : `(:?<first pattern here>)|(:?<second pattern here>)|(:?<third pattern here>)`. It can also be recursively stacked, as seen in the Live Performances example.

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@U2EZNeko commented on GitHub (Jun 17, 2025):

I ended up using this:
(?:(|[|{)[Ll]ive.?(?:)|]|})|[Cc]ommentary|[Rr]e-?master|(?:[Ss]potify|[Rr]adio|[Pp]aramour)\s[Ss]ession|(?:[Ll]ive\s(?:[Ff]rom|[Aa]t|[Ii]n))\s\S

Working fine so far.

EDIT: Pasted old regex

@U2EZNeko commented on GitHub (Jun 17, 2025): I ended up using this: (?:\(|\[|\{)[Ll]ive.*?(?:\)|\]|\})|[Cc]ommentary|[Rr]e-?master|(?:[Ss]potify|[Rr]adio|[Pp]aramour)\s[Ss]ession|(?:[Ll]ive\s(?:[Ff]rom|[Aa]t|[Ii]n))\s\S* Working fine so far. EDIT: Pasted old regex

kerem commented

2026-02-27 04:57:09 +03:00

Author

Owner

@Googolplexed0 commented on GitHub (Jun 17, 2025):

If all seems well, then I'll close the issue and start merging the branch into main (just FYI, the regex matches are case-insensitive, so you don't need to account for that if you want to make your pattern more readable).

@Googolplexed0 commented on GitHub (Jun 17, 2025): If all seems well, then I'll close the issue and start merging the branch into main (just FYI, the regex matches are case-insensitive, so you don't need to account for that if you want to make your pattern more readable).

kerem referenced this issue

2026-02-27 04:57:52 +03:00

[PR #21] [MERGED] Fix oauth not working in docker container #159