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

Problems and requests #39

Open
GoldElysium opened this issue May 21, 2019 · 20 comments
Open

Problems and requests #39

GoldElysium opened this issue May 21, 2019 · 20 comments

Comments

@GoldElysium
Copy link

GoldElysium commented May 21, 2019

Issue 1
AniDB pulls german? data. It should be english as I set that in settings. (#24)

Issue 2
Also, episode data doesn't work correctly. I have it now with Mahouka Koukou no Rettousei and Rakudai Kishi no Cavalry, that all episodes are named the same. The names with both were non existent episode names, with Mahouka it was episode 162 as name and with Rakudai I can't remember but it wasn't correct either. Love Live! did work correctly tho?, everything was called the correct name with the correct description.
This issue is the most annoying for me, manually editting all the metadata is a pain in the ass.

Request
Add Jikan, for MAL support. I use MAL alot, and prefer it above AniDB. Jikan is unofficial but it would be nice if it was added.
The .Net version.

@CTalvio
Copy link

CTalvio commented May 26, 2019

I have not been able to get AniDB to work at all, it either falls back on TVDB or anisearch/proxer (both of which net data in german using this plugin, anisearch also exists in english but this plugin points to the .de version)

Only TVDB or AniDb are supported for episode titles/metadata so the error is cause by some of your series being matched to TVDB correctly and some not. For it to work correctly the filenames seem to have to include season and episode number (most anime releases do not) and for them to correctly parsed, when they are not things get freaky and you get all the episodes detected as episode 1.

ThatNerdyPikachu added a commit to ThatNerdyPikachu/jellyfin-plugin-anime that referenced this issue May 26, 2019
@ThatNerdyPikachu
Copy link

@CTalvio Can you see if the PR above at least solves your Anisearch issue?

@CTalvio
Copy link

CTalvio commented Jun 2, 2019

@ThatNerdyPikachu Sorry, I don't have this figured out enough where I could take your changes all the way to a usable .dll. Provide that though and I'd be happy to test for you, not a dev of any kind myself, just here to be a datapoint when needed.

@Matt07211
Copy link
Contributor

For issue 2 in your post this may be relevant, #33

And as for issue 1 I would guess that since episode metadata isn't getting pulled its falling back to the German metadata providers.

As for MAL I use it a lot to, but until they add back an official API it doesn't seem like a good idea, #7 (comment)

@GoldElysium
Copy link
Author

Ok, I understand that about MAL. I recently switched to Kitsu, still using MAL and anidb next to it. Kitsu has an api. So maybe this is an option to add? Most of the anime have episode descriptions and things like that, some are missing but if it keeps falling back to german this is better than nothing.

@Matt07211
Copy link
Contributor

Yeah gotta wait for someone to implement it, I can certainly give it a go with no promises (mainly because I don't know what in doing and doing it all blind)

@GoldElysium
Copy link
Author

@ThatNerdyPikachu
Can't get it to build:
CS0117: 'HttpRequestOptions' does not contain a definition for 'ResourcePool' Jellyfin.Plugin.Anime
Haven't worked with .NET in a long time, could you maybe build it and send it as a download link?

@bobberb
Copy link

bobberb commented Jul 20, 2019

@ThatNerdyPikachu

same build failure here.

@GoldElysium
Copy link
Author

GoldElysium commented Aug 7, 2019

Looks like #41 got merged with release v4, I'll check tomorrow if this fixes issue 1(and maybe also 2). I would still like to see a Kitsu api implementation if possible, can someone try to code this?

@oddstr13
Copy link
Contributor

oddstr13 commented Aug 8, 2019

Feature requests go here; https://features.jellyfin.org/
Makes it easy to vote and see what's popular.

@GoldElysium
Copy link
Author

Feature requests go here; https://features.jellyfin.org/
Makes it easy to vote and see what's popular.

Oh ok, thanks for pointing that out!

@GoldElysium
Copy link
Author

Hmm, anidb doesn't give any episode metadata, It doesn't give an description and I also still have the issue of that it's pulling the incorrect episode metadata.

@CTalvio
Copy link

CTalvio commented Aug 8, 2019

@GoldElysium
How are you testing the plugin? I have a separate library with files for the monogatari series (since that is one of the most difficult series to pull metadata for) and only AniDB enabled. Even with version 4 NO metadata is pulled from aniDB. The library remains blank with just folder names and filenames as entries.

My main libraries are sort functional with metadata from just TVDB and the few that work from this plugin. As far as I can tell AniDB agent is still completely nonfunctional.

Also, the episode names are because the JF is detecting them as all the same episode, not because of the plugin, you need to change the filenames to correct how they are being detected or do the file move dance (remove series from library, readd) to refresh them.

@zepx
Is there something I am missing?

@GoldElysium
Copy link
Author

GoldElysium commented Aug 8, 2019

@GoldElysium
How are you testing the plugin? I have a separate library with files for the monogatari series (since that is one of the most difficult series to pull metadata for) and only AniDB enabled. Even with version 4 NO metadata is pulled from aniDB. The library remains blank with just folder names and filenames as entries.

My main libraries are sort functional with metadata from just TVDB and the few that work from this plugin. As far as I can tell AniDB agent is still completely nonfunctional.

Also, the episode names are because the JF is detecting them as all the same episode, not because of the plugin, you need to change the filenames to correct how they are being detected or do the file move dance (remove series from library, readd) to refresh them.

@zepx
Is there something I am missing?

I do it the same way you do, having only AniDB enabled. This is how two of my folders look(both aren't getting pulled correctly):
image
image

I think this should be enough in the file name. How do you have your files named?

Also, I posted the request on the site now. link

@CTalvio
Copy link

CTalvio commented Aug 8, 2019

@GoldElysium
All of my files follow the SxxExx naming convention, it seems to be the most reliably detected. I had tons of series doing the same thing you are describing that were simply numbered.

Your examples both fail to match the ones given in them filename guide. The only supported way to give a single digit for episode number is having it at the very start of the filename: https://web.archive.org/web/20181216120039/https://github.com/MediaBrowser/Wiki/wiki/TV-naming

This link is given when viewing the library management window, yes it's an archive link, this info is completely missing from the actual wiki.

Edit: note that if at any point you had TVDB enabled you won't be able to tell if new metadata is pulled from AniDB. Are you getting metadata populated on V4 on a blank library with just AniDB?

@zepx
Copy link
Contributor

zepx commented Aug 8, 2019

I just tested the plugin by creating a library from scratch. AniDB's metadata are indeed populating my Anime series. For starters, I suggest clearing the cache file created by the previous version of this plugin, then rescan your library.

rm -rf <location of jellyfin>/cache/anidb

The pull request solves a major issue with downloading a cache file anidb.

@CTalvio
Copy link

CTalvio commented Aug 8, 2019

@zepx
Brilliant! That did it, we will want to spread that info to users.
I remember this issue also appeared and was similarly solved with the HamaTV plugin for plex, thanks a bunch!!

Episode and season metadata seems to still be failing, filenames are still being displayed, though with this fix already be able to far better get my library together.

With this, might it be time to close this issue?

@zepx
Copy link
Contributor

zepx commented Aug 8, 2019

I will take a look at the episode and season metadata when I have time, I didn't realized that it's not pulling those correctly.

@GoldElysium
Copy link
Author

@GoldElysium
All of my files follow the SxxExx naming convention, it seems to be the most reliably detected. I had tons of series doing the same thing you are describing that were simply numbered.

Your examples both fail to match the ones given in them filename guide. The only supported way to give a single digit for episode number is having it at the very start of the filename: https://web.archive.org/web/20181216120039/https://github.com/MediaBrowser/Wiki/wiki/TV-naming

This link is given when viewing the library management window, yes it's an archive link, this info is completely missing from the actual wiki.

Edit: note that if at any point you had TVDB enabled you won't be able to tell if new metadata is pulled from AniDB. Are you getting metadata populated on V4 on a blank library with just AniDB?

That did it, thanks! Yeah I'm doing it on an empty library.

@zepx
Brilliant! That did it, we will want to spread that info to users.
I remember this issue also appeared and was similarly solved with the HamaTV plugin for plex, thanks a bunch!!

Episode and season metadata seems to still be failing, filenames are still being displayed, though with this fix already be able to far better get my library together.

With this, might it be time to close this issue?

It's indeed only giving the filenames for me too

@ThatNerdyPikachu
Copy link

Bump, still an issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

7 participants