Skip to content
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

[Defaults]: People Defaults Poster and Contents Mis-Match #2112

Open
McBrideMusings opened this issue Jun 13, 2024 · 7 comments
Open

[Defaults]: People Defaults Poster and Contents Mis-Match #2112

McBrideMusings opened this issue Jun 13, 2024 · 7 comments
Assignees

Comments

@McBrideMusings
Copy link

McBrideMusings commented Jun 13, 2024

Link to File

https://gist.github.com/McBrideMusings/248605c83bbb72fb73d7c483da91e140

Version Number

2.0.1

What branch are you on?

master

Describe the Bug

When using the People/Director Default collection the director John Waters TMDB correctly references the director's work but uses the actor John Waters TMDB portrait for the poster.

Screenshot 2024-06-13 at 10 23 50 AM
@bullmoose20
Copy link
Contributor

Link to File

https://gist.github.com/McBrideMusings/248605c83bbb72fb73d7c483da91e140

Version Number

2.0.1

What branch are you on?

master

Describe the Bug

When using the People/Director Default collection the director John Waters TMDB correctly references the director's work but uses the actor John Waters TMDB portrait for the poster.

Screenshot 2024-06-13 at 10 23 50 AM

Can you share the proper link on tmdb for this person so we can adjust please?

@McBrideMusings
Copy link
Author

@YozoraXCII
Copy link
Contributor

@bullmoose20 has anything been done for this? Can this be closed?

@bullmoose20
Copy link
Contributor

So the issue is that there are two legitimate people with the same name... John Waters.
Our repo is based on names.... so Kometa looks for "John Waters"" and finds the only John Waters in our repo... which is the Actor: https://github.com/Kometa-Team/People-Images-rainier/tree/master/J

So when the Director is called, Kometa looks for John Waters and finds the John Waters in the repo which is the Actor and then applies it as the Director.

We could create the Director as "John Waters" and call the file John Waters2

This has been a long standing issue for years in terms of duplicate names.

The team had implemented an "offset" flag to try and address this however the issue is that tmdb will change the search order of the names returned so that sometimes the John Waters you want is not the second one.

@McBrideMusings
Copy link
Author

McBrideMusings commented Aug 14, 2024

That problem makes sense, but based on the config you know that I'm (or any other user looking for a name) is looking for the Director not the Actor. At least SAG requires Actors to have unique names, not sure if the DGA has the same by-law but when I search for this director's name at least I don't see any overlap

https://www.dga.org/The-Guild/Members

I'm not sure how the search is implemented, but could you search both by name and title? At least with Actors that should always work by their own by-laws, and that would likely resolve most of conflicts of the same nature.

Looking at the TMBD API Response, there's a
"known_for_department": "Acting",

field, that returns Directing for directors, seems like that string could be used to properly map the right person based on whether they're in the actor or director default.

also, I don't know if TMBD Ids are supported but if one could plug in the ID directly instead of a name that seems like it would create a workaround for even the most narrow of edge cases

@bullmoose20
Copy link
Contributor

@bullmoose20
Copy link
Contributor

That problem makes sense, but based on the config you know that I'm (or any other user looking for a name) is looking for the Director not the Actor. At least SAG requires Actors to have unique names, not sure if the DGA has the same by-law but when I search for this director's name at least I don't see any overlap

https://www.dga.org/The-Guild/Members

I'm not sure how the search is implemented, but could you search both by name and title? At least with Actors that should always work by their own by-laws, and that would likely resolve most of conflicts of the same nature.

Looking at the TMBD API Response, there's a "known_for_department": "Acting",

field, that returns Directing for directors, seems like that string could be used to properly map the right person based on whether they're in the actor or director default.

also, I don't know if TMBD Ids are supported but if one could plug in the ID directly instead of a name that seems like it would create a workaround for even the most narrow of edge cases

image

Well... I wish it was as simple as "Hey I'm asking for the director when I am using the Directors default from Kometa". Why? because use the Tom Holland example pasted above... 3 actors and 1 director. How would we know which Tom Holland Actor is needed by Kometa to create the proper Tom Holland Actor collection?

The fundamental fix would be to leverage some kind of tmdbID which is unique per person. This would require a rewrite of all that logic within Kometa. And even then, from what I understood, Kometa asks Plex for the cast of people in a movie to come up with the collections and Plex returns NAMES and not IDs (I'd like either @chazlarson or @meisnate12 to confirm this logic). As such, we can only work with names when querying tmdb or our repo for the poster image.

https://kometa.wiki/en/nightly/files/updates/?h=offset
tmdb_person_offset is what we implemented to try and address this and like I mentioned above, tmdb will sometimes return a different order so the tmdb_person_offset is not perfect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants