search

rembo10 / headphones

Automatic music downloader for SABnzbd
GNU General Public License v3.0
3.38k stars 600 forks source link

Post-Processing fails with Dev-Branch #913

Closed ghost closed 11 years ago

ghost commented 11 years ago

Log: http://pastebin.com/D39gbAD5

It fails with ALL(!) downloads. Master-Branch works perfectly.

Seems the ID wouldn't be parsed correctly.

Edit: Pfad e:\Musik_unsorted\Sunbeam - Outside World (Remixes)-CDM-1994-Records INT.3d455eb7-1c74-39dc-aed3-33c6291addce

Tested a little bit:

Nothing worked, allways the same error:

Sunbeam - Outside World (Remixes)-CDM-1994-Records INT.3d455eb7-1c74-39dc-aed3-33c6291addce Sunbeam - Outside World (Remixes)-CDM-1994-Records .3d455eb7-1c74-39dc-aed3-33c6291addce Sunbeam - Outside World (Remixes)-CDM-1994-Records 3d455eb7-1c74-39dc-aed3-33c6291addce Sunbeam - Outside World (Remixes)-CDM-1994-Records . 3d455eb7-1c74-39dc-aed3-33c6291addce

Edit2:

Screenshot of LOG for better reading:

http://i.imgur.com/BsSYK.png

It seems that HP correctly recognize the ID but then didn't analyze the Tracks or recognize the correct album. But it has to be inserted anyway, or not?

Same with another realese. Log from Searching until Post-Processing:

http://i.imgur.com/BYqgd.png

rembo10 commented 11 years ago

It seems like its just the wrong album from the logs? Remix version was grabbed (which i'll get a fix up for) On Nov 4, 2012 6:07 AM, "cappa83" notifications@github.com wrote:

Log: http://pastebin.com/D39gbAD5

It fails with ALL(!) downloads. Master-Branch works perfectly.

Seems the ID wouldn't be parsed correctly.

— Reply to this email directly or view it on GitHubhttps://github.com/rembo10/headphones/issues/913.

ghost commented 11 years ago

Nah, got the problem with all downloads not just remixes.

Its not just for the logs, the album stay in status snatched. Also they aren't processed further (Metadata, moving, renaming, etc.)

Look this: http://i.imgur.com/BYqgd.png

Thats the correct album which found an snatched. Same problem :-)

rembo10 commented 11 years ago

No, paste more of the debug logs, so i can see why it's not matching up. That part of the code hasn't changed though, so it might just be unusual timing. You can post process these albums with the master branch???

ghost commented 11 years ago

Okay, will do it after my kids went in bed.

Give me some hours and i'll come back with more infos.

BTW: Sure you have changed the code - With mster branch HP only use Album name and Year for recognition. In Dev-Branch it uses the Album ID which be give to SABNZBD :+1:

rembo10 commented 11 years ago

Ah yeah - what i meant was, I haven't changed the 'verification' part of that code. It seems like it's parsing the mbid ok because it says "found a match in the database" but it looks like the filenames are a bit different. I'll have to see some other albums before i can tell if there's a problem.

Thanks for testing the develop branch and helping me make headphones awesome!

On Sun, Nov 4, 2012 at 11:12 AM, cappa83 notifications@github.com wrote:

Okay, will do it after my kids went in bed.

Give me some hours and i'll come back with more infos.

BTW: Sure you have changed the code - With mster branch HP only use Album name and Year for recognition. In Dev-Branch it uses the Album ID which be give to SABNZBD [image: :+1:]

— Reply to this email directly or view it on GitHubhttps://github.com/rembo10/headphones/issues/913#issuecomment-10052061.

ghost commented 11 years ago

ARGH, you wouldn't believe that (like I'm):

I downloaded the Master and the Dev in seperate folders and copied the config.ini in the folders. After that I tested three releases - two of the logs and another. All three wouldn't work. In both versions.

Then I choose a album which worked yesterday with Master Branch with the Dev Version and ... it worked(?)

I have to say: All works good how it should be.

But: In the Dev you are working with the mbid's. So HP should automatically "try a new version" if the mbid in downloadfolder can't be recognized.

Maybe with enable/disable this in the options.

rembo10 commented 11 years ago

The mbids are added by headphones before sending to sab so i can recognize them during a forced post process

On Nov 4, 2012, at 1:49 PM, cappa83 notifications@github.com wrote:

ARGH, you wouldn't believe that (like I'm):

I downloaded the Master and the Dev in seperate folders and copied the config.ini in the folders. After that I tested three releases - two of the logs and another. All three wouldn't work. In both versions.

Then I choose a album which worked yesterday with Master Branch with the Dev Version and ... it worked(?)

I have to say: All works good how it should be.

But: In the Dev you are working with the mbid's. So HP should automatically "try a new version" if the mbid in downloadfolder can't be recognized.

Maybe with enable/disable this in the options.

— Reply to this email directly or view it on GitHub.