[Rel] HTTP Anidb Metadata Agent (HAMA)

Dev Community Dev/API Corner
1341

@IBNB 404, files do not exist so can’t download
@MElroumi @chrno And there’s a loop in windows “No local custom mapping in dir: ‘E:’”. And since i use linux (synology) i missed it

Code was for changing the custom mapping file to the serie folder so it can also be used by the scanner, as i could not find a way to have the scanner load the agent file from agent data folder nor find log folder location from the agent…

Uploaded fix in github

1342

@ZeroQI No luck Sir. I would venture its possibly worse then before, for the following reasons.

  1. Took much longer before it even for the point where it tries to load the metadata.
  2. Usually when you create a library and select Hama it would take a few seconds and then show below that the config options for Hama. Now it does’t and when you try to click “add library” it just fails. I left it for a couple of mins then was able to add the library, but still not showing the config options for Hama below the agent select drop down.

In the logs on line 46 its complains about the syntaxError. Couple possibly be the reason for its weird behavior now??

https://drive.google.com/open?id=0B0p08Avg8EvUN25OZ2NlOTdjaDA

ps: if any of the buttons on the plex forums worked for me, I would upload the logs here. Multiple browsers across 3 machines and they don’t work. So sorry for having to keep on linking the logs in google drive, but it works

1343

Tried the updated agent but still no luck.
Here are your logs
https://1drv.ms/f/s!Agyc-p0xRQFAoQ4b0NQ8RQmimXWI

Btw, thank you so much for your efforts

1344

@chrno no problem with attaching logs that way. yes the crash explain all that.

The error is at the semicolon IN the comment (after the hash sign)…
Hold on, the backslash is not escaped so it consider the double quote following and hash sign part of the string…

  File "D:\PlexMS\Plex Media Server\Plug-ins\Hama.bundle\Contents\Code\__init__.py", line 742
    while dir and not dir.endswith("/") and not dir.endswith("\"): # and dir is not "/" and not dir.endswith(":\"):
                                                                                                              ^
SyntaxError: invalid syntax

Corrected it at last, things should be much smoother.
Scanner got a similar enough issue corrected this morning
https://github.com/ZeroQI/Hama.bundle/blob/master/Contents/Code/init.py

1345

@ZeroQI and we have a winner. Busy downloading the metadata as I type.

Thank you good Sir.

1346

Hi Zero.

For me with the newest update it is finally working like a charm. :slight_smile:
Thank you very much for fixxing this one. It is also really fast.
I added a few more shows just to be sure, now I will try it on my collection and will come back.

Screen: Dropbox - File Deleted
Logs: Dropbox - File Deleted

1347

@chrno @IBNB At last… Will accelerate further in a further release.
@IBNB looks really good :smiley:

Thanks for your assistance, it is now better for everybody.
Am doing everything remotelly on my box home so not the easiest and the fact last bug impacted only windows surprised me plus the occasionnal syntax error since i did not test enough but the efforts sure are there…

1348

That’s Microsoft for you always a pain…

1349

Finally, it’s working!
Thanks again for your work, man!

1350

Excellent, we have a stable release at last, thanks all for confirming.

1351

@ZeroQI said:
Excellent, we have a stable release at last, thanks all for confirming.

Awesome to hear, I’ll give it a try later, sadly ASS doesn’t appear to be detecting OP/EDs correctly yet, not sure why.

1352

Already corrected. The line to add the offsets had dissapeared

1353

@ZeroQI said:
Already corrected. The line to add the offsets had dissapeared
Absolute-Series-Scanner/Scanners/Series/Absolute Series Scanner.py at master · ZeroQI/Absolute-Series-Scanner · GitHub

Super. Except trust my “problematic anime” test folder to throw up another spanner. The files are actually “OP1” “OP2” and “OP3” (Coalgirls decided to skip adding OP1b) so it looks like the scanner is confused. That or AniDB doesn’t provide enough data for the scanner to realise that an OP has versions.

1354

Scanner supports op1a etc but files need to be present or it will consider they don’t exist and jump to the next number. Pic above look like perfect numbering

1355

@ZeroQI said:
Scanner supports op1a etc but files need to be present or it will consider they don’t exist and jump to the next number. Pic above look like perfect numbering

So since OP1b doesn’t exist, it just puts OP2 in it’s place? I’m confused since what you’re seeing is not actually correct

1356

The scanner decide season and ep number. 1b is not a proper ep number but i codded bits to get by but it relies on all files being there…
The meta is done by the agent. It expect this ep number to be op1b.
Since op1b is not there is possibly is op2.
The scanner log would show the ep mapping.

1357

I noticed that HAMA doesn’t download poster images of the episodes and uses generated thumbnails instead. Even though I checked “Fetch Poster images from TVDB”. It also uses the show poster image for the season poster image instead of downloading the specific season poster from TVDB.

1358

Yes i have noticed that. It seem to be a plex bug and not an hama bug as the metadata is downloaded and attached at the same time and the function hasn’t changed…

If i am correct even old versions of the agent that were working previously should behave the same.

1359

@ZeroQI said:
Yes i have noticed that. It seem to be a plex bug and not an hama bug as the metadata is downloaded and attached at the same time and the function hasn’t changed…
Plex Server v1.3.x & 3rd Party Agents · Issue #98 · ZeroQI/Hama.bundle · GitHub

If i am correct even old versions of the agent that were working previously should behave the same.

Correct @ZeroQI . We’re tracking it here.

1360

@ZeroQI @MElroumi
Yep, just tested this myself, switching from TheTVDB to HamaTV in a series match means it uses generated thumbnails instead (You can see in this shot they’re TV episodes with a watermark)


(some interesting picks for episode pictures o_O)

Annoying, but since the metadata is there, there’s not much to do except wait for a fix. Sadly the link @sven-7 gave is pass-members only. Kind of dumb for transparency to stick your bug tracking behind a paywall.