search

ankenyr / jellyfin-youtube-metadata-plugin

Youtube Metadata Plugin for Jellyfin
GNU Affero General Public License v3.0
510 stars 33 forks source link

Wrong Metadata #53

Closed ddubnyk closed 2 years ago

ddubnyk commented 2 years ago

It seems with the new release on Jellyfin 10.8 the plugin is matching incorrectly. I've tried 10+ different videos today alone and they are all matched incorrectly. Previous released worked fine.

ankenyr commented 2 years ago

Can you provide more details? Is it similar to #47 where they are all the same episode/ metadata?

On Wed, Jun 15, 2022, 12:58 PM ddubnyk @.***> wrote:

It seems with the new release on Jellyfin 10.8 the plugin is matching incorrectly. I've tried 10+ different videos today alone and they are all matched incorrectly. Previous released worked fine.

— Reply to this email directly, view it on GitHub https://github.com/ankenyr/jellyfin-youtube-metadata-plugin/issues/53, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALYWLWVYWHTORX2XHVPEG3VPIYXXANCNFSM5Y4PIS5Q . You are receiving this because you are subscribed to this thread.Message ID: @.***>

ddubnyk commented 2 years ago

Can you provide more details? Is it similar to #47 where they are all the same episode/ metadata?

On Wed, Jun 15, 2022, 12:58 PM ddubnyk @.***> wrote:

It seems with the new release on Jellyfin 10.8 the plugin is matching incorrectly. I've tried 10+ different videos today alone and they are all matched incorrectly. Previous released worked fine.

— Reply to this email directly, view it on GitHub https://github.com/ankenyr/jellyfin-youtube-metadata-plugin/issues/53, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALYWLWVYWHTORX2XHVPEG3VPIYXXANCNFSM5Y4PIS5Q . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hello, so it's downloading metadata from the right channel, but it's not for the right videos. And they are not all the same episode/metadata. Is there a place a can look at the logs for the plugin in particular by any chance? I have no problem providing them.

ankenyr commented 2 years ago

Check the jellyfin official support for how to pull logs. You may wish to turn in debug logging first and run the metadata stuff again https://jellyfin.org/docs/general/administration/troubleshooting.html

On Wed, Jun 15, 2022, 1:40 PM ddubnyk @.***> wrote:

Can you provide more details? Is it similar to #47 https://github.com/ankenyr/jellyfin-youtube-metadata-plugin/issues/47 where they are all the same episode/ metadata?

On Wed, Jun 15, 2022, 12:58 PM ddubnyk @.***> wrote:

It seems with the new release on Jellyfin 10.8 the plugin is matching incorrectly. I've tried 10+ different videos today alone and they are all matched incorrectly. Previous released worked fine.

— Reply to this email directly, view it on GitHub

53

https://github.com/ankenyr/jellyfin-youtube-metadata-plugin/issues/53, or unsubscribe

https://github.com/notifications/unsubscribe-auth/AALYWLWVYWHTORX2XHVPEG3VPIYXXANCNFSM5Y4PIS5Q . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hello, so it's downloading metadata from the right channel, but it's not for the right videos. And they are not all the same episode/metadata. Is there a place a can look at the logs for the plugin in particular by any chance? I have no problem providing them.

— Reply to this email directly, view it on GitHub https://github.com/ankenyr/jellyfin-youtube-metadata-plugin/issues/53#issuecomment-1156909223, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALYWLWS6XRAOAL5CCP2M5DVPI5VRANCNFSM5Y4PIS5Q . You are receiving this because you commented.Message ID: @.***>

ddubnyk commented 2 years ago

This is the error I'm getting in the log.

[2022-06-15 21:25:02.655 +00:00] [ERR] [56] MediaBrowser.Providers.Movies.MovieMetadataService: Error in "YoutubeMetadata" System.IO.FileNotFoundException: youtube-dl not found on path! at NYoutubeDL.Services.PreparationService.SetupPrepare(YoutubeDL ydl) at NYoutubeDL.Services.InfoService.GetDownloadInfoAsync(YoutubeDL ydl, CancellationToken cancellationToken) at NYoutubeDL.Services.PreparationService.PrepareDownloadAsync(YoutubeDL ydl, CancellationToken cancellationToken) at NYoutubeDL.Services.DownloadService.DownloadAsync(YoutubeDL ydl, CancellationToken cancellationToken) at NYoutubeDL.Services.DownloadService.DownloadAsync(YoutubeDL ydl, String url, CancellationToken cancellationToken) at NYoutubeDL.YoutubeDL.DownloadAsync(String videoUrl) at Jellyfin.Plugin.YoutubeMetadata.Utils.YTDLMetadata(String id, IServerApplicationPaths appPaths, CancellationToken cancellationToken) at Jellyfin.Plugin.YoutubeMetadata.Providers.YTDLMovieProvider.GetAndCacheMetadata(String id, IServerApplicationPaths appPaths, CancellationToken cancellationToken) at Jellyfin.Plugin.YoutubeMetadata.Providers.AbstractYoutubeRemoteProvider3.GetMetadata(E info, CancellationToken cancellationToken) at MediaBrowser.Providers.Manager.MetadataService2.ExecuteRemoteProviders(MetadataResult1 temp, String logName, TIdType id, IEnumerable1 providers, CancellationToken cancellationToken)

ddubnyk commented 2 years ago

Actually I'm seeing that my API key disappeared. Ill add that back in and check again.

Edit: That didn't help and i think i remembered incorrectly. Don't believe this plugin needs an API key.

ankenyr commented 2 years ago

You need yt-dlp for it to work now. The API has some limitations which prevent it from being useful for larger libraries. I removed it to focus on the yt-dlp downloader.

On Wed, Jun 15, 2022, 2:46 PM ddubnyk @.***> wrote:

Actually I'm seeing that my API key disappeared. Ill add that back in and check again.

— Reply to this email directly, view it on GitHub https://github.com/ankenyr/jellyfin-youtube-metadata-plugin/issues/53#issuecomment-1156978057, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALYWLQEWPN3XR55N5V7TETVPJFMJANCNFSM5Y4PIS5Q . You are receiving this because you commented.Message ID: @.***>