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

Add redirect for Sorairo Utility #189

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FichteFoll
Copy link
Contributor

@FichteFoll FichteFoll commented Jan 25, 2025

The TV series from 2025 has the same name as the OVA of 2021 and is disambiguated by a "(TV)" suffix sometimes, but this is unreliable and it's is much more likely that the series is accidentally recognized as the OVA. Since the OVA only has one episode, we can redirect all other episodes to the TV series.

(Kitsu doesn't even show the (TV) disambiguator and only has it in the URL: https://kitsu.app/anime/sorairo-utility-tv.)

The TV series from 2025 has the same name as the OVA of 2021 and is
disambiguated by a "(TV)" suffix sometimes, but this is unreliable and
its is much more likely that the series is accidentally recognized as
the OVA. Since the OVA only has one episode, we can redirect all other
episodes to the TV series.
@erengy
Copy link
Owner

erengy commented Feb 15, 2025

Thanks for the explanation, but is this necessary?

Sorairo Utility - 01.mkv is going to be ambiguous in any case.

Sorairo Utility - 02.mkv and onwards don't need a redirection because applications already know that the OVA has only 1 episode. Comparing the episode number with total episodes (e.g. 2 <= 1), if available, should be enough to determine that the OVA cannot be a match. This is what Taiga does, for instance.

@FichteFoll
Copy link
Contributor Author

FichteFoll commented Feb 16, 2025

Trackma does an initial title match first and afterwards performs episode redirects. The title matching logic is not affected by the maximum episode count of the matched show, as far as I'm aware, so it would matter there.

I could double-check this, but having such a redirect wouldn't hurt either way.

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

Successfully merging this pull request may close these issues.

2 participants