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

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

Closed
atraholt opened this issue Oct 20, 2018 · 5 comments
Closed

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

atraholt opened this issue Oct 20, 2018 · 5 comments

Comments

@atraholt
Copy link
Contributor

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
Copy link
Owner

ScudLee commented Oct 20, 2018

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
Copy link
Contributor Author

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
Copy link
Owner

ScudLee commented Oct 20, 2018

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
Copy link
Contributor Author

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
Copy link
Owner

ScudLee commented Oct 20, 2018

Fixed in 43f1b6d (misnamed the commit)

@ScudLee ScudLee closed this as completed Oct 20, 2018
mikethecalamity pushed a commit to mikethecalamity/anime-lists that referenced this issue Jul 6, 2022
* added some tvdb ids

* Update anime-list-master.xml
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

2 participants