Metadata

Hey,

Cheers for adding metadata, Couple of questions.

Are we able to force it to download missing metadata from our collection? I did a full library update and only tvshow.nfo was downloaded for some shows. No images, episode metadata or episode images downloaded.

Also with the tvshow.nfo I think the thumb has a bug as seen here, Should it be referring to NzbDrone.Core.MediaCover.MediaCover

`

A Touch of Cloth 79 Hannah plays DI Jack Cloth, who is called in to investigate an apparent series of serial killings alongside his new partner, DC Anne Oldman, described as a "plucky, no-nonsense sidekick". Playing with the cliches and conventions of British police dramas, subplots include Cloth dealing with visions of his dead wife and the bisexual DC Oldman coming to grips with her feelings for both her female fiancee and Cloth. 260750 Comedy 2012-08-26 Sky1 Shintey NzbDrone.Core.MediaCover.MediaCover`

Thanks again!

Looks like the forum removed the xml tags.

Here is a screenshot of one of my tags.

Also noticed some of the show images have downloaded but still no episode meta for existing shows

Mine mentions the following:
Generating \\cuba\Videos\TV Shows\Girls\Season 3\Girls - S03E07 - Beach House.nfo for: \\cuba\Videos\TV Shows\Girls\Season 3\Girls - S03E07 - Beach House.mp4

but no such file is ever created

After a reboot of the service it is working correctly, can also confirm <thumb></> is being created fine instead of the variable Tzortst is seeing.

that was thumb in tags, forum deleted it/ formatted it.

I restarted my machine, also deleted the tvshow.nfo and it is still inserting NzbDrone.Core.MediaCover.MediaCover for Thumb

Also looking at the logs getting this

System.InvalidOperationException: Sequence contains more than one matching element
at System.Linq.Enumerable.SingleOrDefault[TSource](IEnumerable1 source, Func2 predicate)
at NzbDrone.Core.Metadata.Consumers.Xbmc.XbmcMetadata.WriteTvShowNfo(Series series, List1 existingMetadataFiles) in m:\\BuildAgent2\\work\\328d72309b633a8\\src\zbDrone.Core\\MetaData\\Consumers\\Xbmc\\XbmcMetadata.cs:line 233 at NzbDrone.Core.Metadata.Consumers.Xbmc.XbmcMetadata.OnSeriesUpdated(Series series, List1 existingMetadataFiles) in m:\BuildAgent2\work\328d72309b633a8\src\zbDrone.Core\MetaData\Consumers\Xbmc\XbmcMetadata.cs:line 57
at NzbDrone.Core.Metadata.NotificationService.Handle(MediaCoversUpdatedEvent message) in m:\BuildAgent2\work\328d72309b633a8\src\zbDrone.Core\MetaData\MetadataService.cs:line 35
at NzbDrone.Core.Messaging.Events.EventAggregator.PublishEvent[TEvent](TEvent event) in m:\BuildAgent2\work\328d72309b633a8\src\zbDrone.Core\Messaging\Events\EventAggregator.cs:line 56

I should add that metadata for new episodes are downloading correctly, both image and nfo and the nfo seems to have the correct image path for the thumb.

Seems to only be an issue with the tvshow.nfo (the overall tvshow nfo)

Also still be nice to know if there is any way to force it to download metadata for existing shows/episodes :slight_smile:

@tzortst Found the issue with the actor URL. The error was due to season images being stored in the database with the wrong type (stored as series metadata instead of season image).

I have pushed fixes for both these issues, which are available in 2.0.0.1141

Right now episode metadata isn’t created when the series refresh runs, just when files are imported (and renamed when files are renamed).

Also you can use back ticks (\) around XML/other code to put it in a code block like so:`. Edit is also your friend instead of double posting.

Updated and all looks good.

Any plan to add a force metadata update for all episodes?

Will also keep note of double posting.

Cheers

An option should be added to this new feature to stop the creation of folders and metadata for series that are not monitored.

The option to create folders is set to off, but apparently this new metadata feature overrides that.

Shedrock

@shedrock Why would you add a new series and not monitor it?

Where in my post does it say that this happens when I add a new series?

I of course have series that are already there which are unmonitored at the moment. An example of this would be the TV series Last Resort. It is unmonitored but with the metadata being active creates a folder called Last Resort.

I ‘moved’ the original folder to another drive because of space. I chose to keep it drone (unmonitored) so that I would remember the path. This is done so that when I do decide to purchase another HD, I can then remember the original path the series was on.

Putting that aside, why would ND create a folder that does not exists, especially if the other option to create series folder names is set to off? And yes, I know that is not related to the metadata feature.

Thank you for your work on the metadata. I presume once stable, you will look into adding a refresh all option?

keep up the good work! :slight_smile:

@tzortst said:
Updated and all looks good.

Any plan to add a force metadata update for all episodes?

Will also keep note of double posting.

Cheers

@shedrock whats with the attitude? It was the specific case I considered, but not the only case. I’ll add it to Trello to not create metadata if the folder doesn’t exist on disk, but monitored status will not be considered. https://trello.com/c/5RIEB3gJ/565-do-not-create-metadata-files-if-series-folder-does-not-exist

@erwintwr yeah, didn’t want to create a mess with all the episode file metadata straight away, especially if it was wrong. Please let us know of any issues. Added to Trello though. https://trello.com/c/rZbdJa4z/564-create-missing-episode-file-metadata-when-series-is-refreshed

Sorry Markus. I sincerely apologize. I should not have responded to the message while experiencing severe neck and shoulder pain.

Again, my sincerest apologies.

Shedrock

No worries Shedrock. Feel better.

Another issue was posted on Trello: https://trello.com/c/O3WFhVEk/573-xbmc-episodes-shouldn-t-use-display-season-epsiode - it is fixed in develop

@tzortst @erwintwr should metadata be updated for each episode when the info is refreshed from trakt? Or just added if its missing? Be nice to have updated data, but not if it causes other unknown issues.

@markus101 updated metadata would be ideal, as sometimes there are mistakes on trakt that are later updated. You thinking of pushing any change from trakt straight through to the local metadata?

This shouldn’t have any issues with XBMC. My understanding is once XBMC has scanned the nfo file it won’t scan it again unless you force a full refresh but no reason why the local metadata shouldn’t have the correct information.