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

Using anitopy library for parsing names #6

Closed
ghost opened this issue Jun 9, 2023 · 2 comments
Closed

Using anitopy library for parsing names #6

ghost opened this issue Jun 9, 2023 · 2 comments

Comments

@ghost
Copy link

ghost commented Jun 9, 2023

I think Anitopy is unmatched when it comes to parsing names, and it would be useful for this project I think, and its easy to use.

@hotsno
Copy link
Owner

hotsno commented Jun 14, 2023

Thanks for sharing!

But I've actually known about this even before I started working on Keroro. This unfortunately doesn't do the (in my opinion) hardest part of mapping file names to entries on AniList, which is finding the correct AniList entry given the episode number, season #, and anime name as it appears in the file name. The last part of that sentence is the important part.

Using the information as it appears in the file name and then querying AniList for the top result with that is going to be inaccurate a lot of the time. I've looked at how some other projects handle this, and it's not something I want to invest my time and energy into. I could maybe use someone else's code as reference, but the one that I looked at was in JavaScript and involves so much potentially flaky logic that I wouldn't feel good using it without writing unit tests because I'd probably make several mistakes while rewriting it. This would add even more time and effort, given that it actually requires me to understand what all of this pretty tedious logic is doing.

Unfortunately, I don't think I'll ever be automating file name -> AniList entry mapping with Keroro, even though I would have liked for it to have that feature.

I've personally actually found manually mapping anime names to not be as annoying as I thought I would, and this is coming from someone who loves automation. It also guarantees it's always accurate.


If by chance anyone ever wants to work on this feature, feel free to leave a comment here beforehand, but most I'll almost certainly be open to merging a PR for this!

@ghost
Copy link
Author

ghost commented Jun 14, 2023

Oh I see thanks for detailed answer also thanks for developing this awesome project.

@ghost ghost closed this as completed Jun 14, 2023
This issue was closed.
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

1 participant