-
Notifications
You must be signed in to change notification settings - Fork 74
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
TMDB integration #1190
Comments
Run import action |
Thanks, will try it. Ragarding the matching issue EP 03 not using S01E03 I downloaded the newest daily/canary build and yes, the issue was fixed. |
Webui uses the anidb image while Metadata adds TMDB/TVDB images first and then AniDB |
Had a look at the *.py file of shoko metadata and found the following lines of code in
I double checked with swagger, what the order of images is when this api is called The order seems to be correct, first ANIDB and than TMDB, so I diged further and found this line of code:
the method metadata_add looks like this:
the code seems to work with indexes, in the code:
but looking at the swagger result the indexes seems to be correct. idx = try_get(art, 'index', 0)
+ idx += 1
Log("[metadata_add] :: Adding metadata %s (index %d)" % (url, idx)) I verified that this line is working as one would expect by doing this steps:
to double check, I did those step
Note: I would assume this to be an error, which also present in the current master of shoko metadata ping @Cazzar , maybe you want to chime in |
I would make an issue on that repo so that it doesn't go missing |
Noticed the drop of TvDB, I had auto-link enabled sadly this setting didn't get ported to TMDB.
Therefor I noticed new episodes didn't get images as it used to.
I re-enabled the auto-link setting, but now I wonder is there a way to tell shoko to try to automagically match my entire collection to TMDB?
Notice a bug (or unknown behaviour) in matching episodes as well.
Why did EP 03 not match with S01E03 the date is identical, S01E01 matches neither title nor date
Also should BG (Best Guess) exclude entries that already have DT (Dates and Title Match) ?
Edit:
Saw the latest commits, so maybe the matching was already fixed.
The text was updated successfully, but these errors were encountered: