ScudLee / anime-lists

150 stars 105 forks source link

"(Gekijouban) Shakugan no Shana" specials mapping #150

Closed atraholt closed 6 years ago

atraholt commented 6 years ago

This one's a bit of a trickster.

TheTVDB maps The Sovereign Carmel-san, s0e05. https://www.thetvdb.com/series/shakugan-no-shana/seasons/0 AniDB considers this a T for the movie, Gekijouban Shakugan no Shana. and maps it as T5 and T6. In all technicality, these two things aired back to back with a commercial break between them. https://anidb.net/perl-bin/animedb.pl?show=anime&aid=4388

I'm not sure Ts can be mapped?

ScudLee commented 6 years ago

In the Kodi scraper, Ts are treated as specials starting at 201+, so it is possible to map them. Other apps might not recognize that though.
The best you can do in this case is map the multiple anidb episodes to the same tvdb episode, so they'll all have the same tvdb plot and thumbnail.

My read on the tvdb plot summary, though, suggests that it also includes T4 ("Shakugan no Shana The Movie Promotional Video"). Do you agree?

atraholt commented 6 years ago

Not sure if HAMA in Plex will recognize that, which is the one I use. The Kodi scraper gets you banned from AniDB, so I daren't use it.

T4, if I recall correctly, was on air before the movie was in theatres. T5 and T6, which was the one featuring Carmel, aired after the movie was in theatres, further promoting it.

ScudLee commented 6 years ago

Having not got any of the files to check (or even watched Shana past the first season), I'll defer to your judgement on T4.

As for HAMA, you'll have to ask over there how they handle trailer mappings. (There are, by my count, two other instances already in the mapping list.)

The full fix will be:

<anime anidbid="4388" tvdbid="78962" defaulttvdbseason="0" episodeoffset="3" imdbid="tt0994794">
    <name>Gekijouban Shakugan no Shana</name>
    <mapping-list>
      <mapping anidbseason="0" tvdbseason="0">;1-6;2-0;3-0;4-0;6-0;7-0;205-5;206-5;</mapping>
    </mapping-list>
  </anime>
atraholt commented 6 years ago

Also worth mentioning that importing the equivalent special from MyAnimeList also only marked T5 and T6 on AniDB, so I think we've got it right.

ScudLee commented 6 years ago

Fixed in 43f1b6d40e0913e3040d3a5ddc0ecbcbe90c7e26 (misnamed the commit)