search

Anime-Lists / anime-lists

118 stars 53 forks source link

Removed wrong mappings #391

Closed Guslletas closed 1 year ago

Guslletas commented 1 year ago
AniDB TVDB/TMDB/IMDB Notes
https://anidb.net/anime/708 https://thetvdb.com/index.php?tab=series&id=79654 Removed mapping
https://anidb.net/anime/3626 https://thetvdb.com/index.php?tab=series&id=286895 Removed mapping

So I've been implementing support for mapping using absolute numbers on my app and found those 2 are wrongly mapped:

Those two seem like a nightmare to map manually but I believe it's better to not provide the mappings than to provide wrong ones.

BrutuZ commented 1 year ago

The IDs exist and match on both. The TVDB entry being incomplete or "too complete" is out of scope for the list, it's just a matter of some dedicated volunteer fixing up the TVDB entries

Guslletas commented 1 year ago

The IDs match but the mappings are wrong, it's not about the tvdb entry being incomplete it's about the mappings providing wrong info. For those 2 shows the absolute numbering mapping can't be used because TVDB's absolute order and AniDB one can't match. For example according to the mappings for Shin Chan anidb's 1006 episode(https://anidb.net/episode/219235) is mapped to tvdb absolute order episode 1006(https://thetvdb.com/series/crayon-shin-chan/episodes/6871930) when it should map to tvdb's episode 1062(https://thetvdb.com/series/crayon-shin-chan/episodes/7311859).

BrutuZ commented 1 year ago

Still no reason to remove the ID mapping. If there's some discrepancy midway it should be fixed either on TVDB or with a mapping-list override