RandomNinjaAtk / docker-lidarr-extended

lidarr-extended :: Lidarr application packaged with multiple scripts to provide additional functionality
GNU General Public License v3.0
276 stars 24 forks source link

Compilation Album Unsuccessful, Numbered Versions Downloaded, Audio Script Hangs #136

Closed Bait-Fish closed 1 year ago

Bait-Fish commented 1 year ago

Describe the bug

With album named "Relaxing Bossa Lounge" monitored in Lidarr-Extended, Audio script v1.0.292 did not download that specific album. Many numbered variations of Relaxing Bossa Lounge XX successfully downloaded. No version of Relaxing Bossa Lounge imported to LE.

Audio script eventually stopped functioning. Found script hung/not functioning the following day.

To Reproduce Steps to reproduce the behavior:

Container setting "enableVideoScript" set to "false", with "Relaxing Bossa Lounge" already monitored in Lidarr-Extended (2008 compilation)

Expected behavior Expected to download a single album

Log Data https://pastebin.com/K6WJdtsx Note, middle of the log removed due to Pastebin size limitation

Docker Info:

Additional context

This issue did not present when container setting "enableVideoScript" to "true". Suspect the video script caused 'waiting for video download' bypassed the album in that case

RandomNinjaAtk commented 1 year ago

Just because something is downloaded, does not guarantee a successful import, since it's up to lidarr to make the final decision...

Also what your seeing is expected behavior, if you want to resolve it, you should update musicbrainz with a link for the album: https://github.com/RandomNinjaAtk/docker-lidarr-extended/wiki/Extended-Audio-Script-Information#q-it-cannot-find-albums-that-i-know-should-be-available

Edit: Also "Various Artists" albums are very difficult to handle in general, so you shouldn't expect much success with them, unless you've added the links to musicbrainz.

2nd Edit:

Your additional context may be valid, I'm not sure I accounted for VA albums when videos are enabled. I'll have to look into that.

RandomNinjaAtk commented 1 year ago

Your additional context note should be fixed in 1.0.296, but that is the only bug identified here...

Bait-Fish commented 1 year ago

Thanks. Will check.

Bait-Fish commented 1 year ago

I have not been able to check this while focused on other troubleshooting.

RandomNinjaAtk commented 1 year ago

@Bait-Fish Is this resolved?

RandomNinjaAtk commented 1 year ago

Closing due to lack of feedback, please open a new issue if still having a problem...