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

Better handling for trailers, extras & bonus material for movies and TV series #154

Closed
Omertron opened this issue Mar 15, 2015 · 24 comments
Closed

Comments

@Omertron
Copy link
Member

Original issue 154 created by Omertron on 2008-08-25T05:20:21.000Z:

How can we handle the Extras stuff included on DVDs, or special reports for
TV series in each season.

One solution is to add the ability to parse s01x01 to mean extra 1 of season 1.
That way it won't be confused with s01e01 for the actual episodes.

Right now for YAMJ I have to add Extras/ directory to the list of exclusions
since most of the file names do not include the series name or season number.

@Omertron
Copy link
Member Author

Comment #1 originally posted by Omertron on 2008-10-01T10:25:13.000Z:

<empty>

@Omertron
Copy link
Member Author

Comment #2 originally posted by Omertron on 2008-10-22T15:44:34.000Z:

I would also love ways to handle extra material both for movies and series!

@Omertron
Copy link
Member Author

Comment #3 originally posted by Omertron on 2008-10-25T00:18:54.000Z:

This is a great idea. I have many movies with extras such as "making of" and cast
interviews that I would like to see on the same detail page as the movie. Please
consider this.

@Omertron
Copy link
Member Author

Comment #4 originally posted by Omertron on 2008-10-25T00:19:10.000Z:

This is a great idea. I have many movies with extras such as "making of" and cast
interviews that I would like to see on the same detail page as the movie. Please
consider this.

@Omertron
Copy link
Member Author

Comment #5 originally posted by Omertron on 2008-10-26T00:29:51.000Z:

I think besides using s01x01 as extra for season1 or s01e01x01 as extra for ep1 of
season 1,
we could instead be using the trailer scanner with modifications.

So this could also be used for extras, bloopers, cut-scenes, bonus, etc.
So we could be using this trailer feature for a whole lot more...
moviename[trailer-desc]
moviename[extra-Making of]
moviename[extra-Bloopers 1]
moviename[extra-Bloopers 2]
moviename[bonus-Alternate Ending]
moviename[extra-Deleted Scenes]

So anything after the '-' is a description (leading/trailing spaces removed), and if
no description found as in
moviename[extra]
then just set description to "Extra". One more field needs to be added to the XML
file to state what kind of "trailer" it is,
ie: trailer, extra, bonus, deleted, etc, so that the skin knows how to handle it in
case they should be grouped on a different part of the output.

Or probably better is to use the <file...> and create
<trailer...>
<extra...>
<bonus...>
etc
because each of those will have a count associated to them, if any defined.

Again, that should be the skin's job to handle them.

@Omertron
Copy link
Member Author

Comment #6 originally posted by Omertron on 2008-12-01T01:46:24.000Z:

PLEASE PLEASE add this, when time permits. This would be so very helpful.

Thanks a bunch.

@Omertron
Copy link
Member Author

Comment #7 originally posted by Omertron on 2008-12-01T11:01:42.000Z:

Didn't spot this issue when raising
http://code.google.com/p/moviejukebox/issues/detail?id=306

Here's a copy of my request which differs (slightly) from this issue

"Please rename the Trailers section to Extras and place any entries with
square brackets there eg:-

tv.series.name.s01e01[Christmas Special].avi
tv.series.name.s01e01[Bloopers].avi
tv.series.name.s01e01[anything at all].avi
tv.series.name.s01e01[cut scenes].avi
tv.series.name.s01e01[Extras].avi
tv.series.name.s01e01[with commentary].avi

Users would then be able to write a description exactly as they want it to
appear within the square brackets which is more useful than the page just
saying Trailer1, Trailer2 etc.

There would have to be exceptions for recognized multipart identifiers [CD1],
[Disk1] etc"

@Omertron
Copy link
Member Author

Comment #8 originally posted by Omertron on 2008-12-01T15:38:12.000Z:

@tcpaulh
We more than likely will not accept "anything" between brackets as an Extra. This is
because brackets could be a special delimiter for other keywords. But we will be
examining the usage of various keywords and standardize some naming conventions.

@Omertron
Copy link
Member Author

Comment #9 originally posted by Omertron on 2008-12-01T21:59:25.000Z:

My view is that while introducing more keywords might be a good solution
programmatically, it would not serve to simplify usage which I understand is one of
the major development goals.

I think it would be simpler for the user if [anything] could be entered + a huge plus
would be that the user can write their own description for an entry which would be
displayed on the screen.

I'm struggling to see a need for reserved words ([cd1],[cd2],[trailer] etc) if
anything in square brackets is simply added to what is currently the trailer list.

An alphabetic sort would tend to get things in the right order anyway.

Sorry if I'm missing something obvious.

@Omertron
Copy link
Member Author

Comment #10 originally posted by Omertron on 2008-12-01T22:50:47.000Z:

My view is that while introducing more keywords might be a good solution
programmatically, it would not serve to simplify usage which I understand is one of
the major development goals.

I think it would be simpler for the user if [anything] could be entered + a huge plus
would be that the user is effectively writing their own display description.

Having a short exception list which is tested for [cd1],[disk1],[part1] etc could
still serve to keep entries in the proper place.

Sorry if I'm missing something obvious.

@Omertron
Copy link
Member Author

Comment #11 originally posted by Omertron on 2008-12-01T23:15:36.000Z:

It's for organizational purposes in the skins. If everything is just lumped together,
then the skin developers have no way of knowing what is a trailer or Blooper, or
anything else, in order to display them or not accordingly, and then you'd just end
up with one big list of stuff with no organization.

@Omertron
Copy link
Member Author

Comment #12 originally posted by Omertron on 2008-12-02T00:01:30.000Z:

Everything 'extra' is already lumped together in one place. :S

Could exceptions not still be parsed in the normal way but with any unhandled text
within [] being used as a description?

@Omertron
Copy link
Member Author

Comment #13 originally posted by Omertron on 2008-12-02T05:57:47.000Z:

Having done some skin changes myself, I know what a pain it is to work with and get
things lined up.
As the requester, I have no problem with taking "anything" inside of brackets as the
description, if no keyword is found within it.
I also have no problem with keywords being searched if the user wants to name them
that way: trailer, extra, bonus, blooper, deleted, other.

In post # 5 above I suggested using
Or probably better is to use the <file...> and create
<trailer...>
<extra...>
<bonus...>
etc

And this still seems appropriate and takes care of my wishes, "tcpauth", and the
developers since anything with no tag could be placed in <other...>
and then leave it up to the skin to decide what to do.

But as I said, modifying skins really sucks if you have too many options to check
for. With that, maybe it would be easier for the skin developers if it was:
Or probably better is to use the <file...> and create
<type="trailer" file=...>
<type="extra" file=...>
<type="bonus" file=...>
<type="other" file=...>
etc

@Omertron
Copy link
Member Author

Comment #14 originally posted by Omertron on 2008-12-02T06:08:50.000Z:

Sorry, I forgot to mention that anything with a tag should take anything after the
hyphen/dash '-' as the description. So the last part above should be:
<type="trailer" desc="demo" file=...>
<type="extra" desc="Behind the scenes" file=...>
<type="bonus" desc="Alt Ending" file=...>
<type="other" desc="full contents within square backets" file=...>
etc

As well, these should not just be with the first episode in directory, user can
define per episode special videos and also per season by naming it as directory name.
EX:
New-movie/My-movie.720p.mkv
New-movie/My-movie.720p[bonus-Alt Ending].mkv
New-movie/My-movie.720p[trailer].mkv

TV-show/My-show s02/My-show s02[bonus-Director comments].mkv
TV-show/My-show s02/My-show s02e01.mkv
TV-show/My-show s02/My-show s02e01[deleted-Scene 1].mkv
TV-show/My-show s02/My-show s02e01[deleted-Scene 2].mkv
TV-show/My-show s02/My-show s02e02.mkv
TV-show/My-show s02/My-show s02e03.mkv
TV-show/My-show s02/My-show s02e03[deleted-Scene 1].mkv
etc

@Omertron
Copy link
Member Author

Comment #15 originally posted by Omertron on 2008-12-02T07:49:07.000Z:

I think this is going to have to be taken in as part of the video grouping if you are
not naming the bonus the same as the original file name. E.g. the TV-show/My-show
s02/My-show s02[bonus-Director comments].mkv is going to cause difficulties.

@Omertron
Copy link
Member Author

Comment #16 originally posted by Omertron on 2009-02-18T11:13:37.000Z:

Issue 533 has been merged into this issue.

@Omertron
Copy link
Member Author

Comment #17 originally posted by Omertron on 2009-03-04T13:59:32.000Z:

It is possible to mark movies as parts with titles, extracted from file name:

Moviename Part1.avi
Moviename Part2 - Making of.avi
Moviename Part3 - Bloopers.avi

This will be places on the one movie details screen as movie parts.

@Omertron
Copy link
Member Author

Comment #18 originally posted by Omertron on 2009-03-04T14:02:20.000Z:

See also issue 155

@Omertron
Copy link
Member Author

Comment #19 originally posted by Omertron on 2009-05-05T20:42:06.000Z:

If it matters to anyone, the way I have been handling extras on a DVD / BD is to rip
them separately, then name them whatever I like and use the SET keyword to keep em
all together. I use NFOs judiciously to prevent YAMJ from scraping IMDB for them and
failing. In some cases you get lucky, for example Pumping Iron 25th Anniversary release.

In this case, some of the extras are properly indexed in IMDB.

Either way, here is how I have my example above set up:

O:\Documentaries\Pumping Iron [1977] DVD
Iron and Beyond [2002][SET Pumping Iron-2] DVD.nfo
Iron and Beyond [2002][SET Pumping Iron-2] DVD.png
Iron and Beyond [2002][SET Pumping Iron-2] DVD.VOB
Iron Insights [2002][SET Pumping Iron-4] DVD.nfo
Iron Insights [2002][SET Pumping Iron-4] DVD.png
Iron Insights [2002][SET Pumping Iron-4] DVD.VOB
Pumping Iron Premiere Party [2002][SET Pumping Iron-6] DVD.nfo
Pumping Iron Premiere Party [2002][SET Pumping Iron-6] DVD.png
Pumping Iron Premiere Party [2002][SET Pumping Iron-6] DVD.VOB
Pumping Iron [1977][SET Pumping Iron-1] DVD.jpg
Pumping Iron [1977][SET Pumping Iron-1] DVD.nfo
Pumping Iron [1977][SET Pumping Iron-1] DVD.VOB
Raw Iron The Making of 'Pumping Iron' [2002][SET Pumping Iron-3] DVD.nfo
Raw Iron The Making of 'Pumping Iron' [2002][SET Pumping Iron-3] DVD.png
Raw Iron The Making of 'Pumping Iron' [2002][SET Pumping Iron-3] DVD.VOB

The NFO for those that are not in IMDB have 0 to prevent scraping and then
all pertinent info filled in the NFO

The posters for those extras not covered by scraper are generated from screenshots
via VLC.

Main Grouping
http://manny.smugmug.com/photos/529151178_yD7WW-XL.jpg

Sample Extra
http://manny.smugmug.com/photos/529151214_UMzMW-XL.jpg

@Omertron
Copy link
Member Author

Comment #20 originally posted by Omertron on 2009-07-13T14:18:51.000Z:

I have come up with a work around for this that suits my needs so I thought I'd post
it here as well.

I am using I am Legend as my example.

Naming as follows:

main movie: I Am Legend
Trailer: I Am Legend[Trailer]
Alternate Ending: I Am Legend[Trailer-Alt Ending]

run YAMJ. After it is done running go to your Jukebox folder and search for the main
I Am Legend.html file.
Once you find the file using notepad or a HTML editor edit the file and search for
"Trailer-Alt Ending" (Or whatever your name was) Change the text to
Alternate Ending. Save the file and go in to your jukebox. You will now see on the I
Am Legends page, the Trailer and the Alternate Ending on the page.

@Omertron
Copy link
Member Author

Comment #21 originally posted by Omertron on 2009-07-14T13:01:10.000Z:

I've started work on this, but there is some major re-work required to skins once
it's complete.

All additional files will now be called "extras" including trailers.

Currently I have Trailer/Bonus/Extra, but I'm planning on changing that to be
parameter driven, so a keyword can be used to place a file in the "Extras" category
of a video file.

Seems to be working for TV and Movies, but I've broken categories. As soon as I sort
it, I'll post a version for testing

@Omertron
Copy link
Member Author

Comment #22 originally posted by Omertron on 2009-07-16T09:52:42.000Z:

This is committed as r1037.
WARNING: YOU WILL NEED TO REBUILD YOUR ENTIRE JUKEBOX

There is a new parameter "filename.extras.keywords=trailer,bonus,extra" the controls
the keywords that are searched for in the [] to determine what is considered an extra.

So if you want a new extra type "deleted" you would add that to the above parameter:
"filename.extras.keywords=trailer,bonus,extra,deleted"
and name your file:
moviename.[Deleted-Deleted Scenes].avi
TV Series S01E01 [Deleted-Deleted Scenes Episode 1].avi
TV Series S01E20 [Deleted-Deleted Scenes Episode 20].avi
etc..

@Omertron
Copy link
Member Author

Comment #23 originally posted by Omertron on 2009-07-16T14:45:06.000Z:

This sounds nice =)

@Omertron
Copy link
Member Author

Comment #24 originally posted by Omertron on 2009-07-17T07:41:24.000Z:

<empty>

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

No branches or pull requests

1 participant