YAMJ / yamj-v2

Yet Another Movie Jukebox (YAMJ) v2
GNU General Public License v3.0
28 stars 11 forks source link

Overlay misplaced when mjb.MaxThreadsProcess > 1 #2063

Closed Omertron closed 9 years ago

Omertron commented 9 years ago

Original issue 2064 created by Omertron on 2011-08-14T10:30:24.000Z:

What steps will reproduce the problem?

  1. My settings for thumbnails (I have the same settings and results for posters)

A. Jukebox properties mjb.MaxThreadsProcess = 4

B. Skin user properties thumbnails.width=127 thumbnails.height=178 thumbnails.stretch=true thumbnails.normalize=false thumbnails.xmlOverlay=false thumbnails.overlay=true thumbnails.overlay.source=videosource thumbnails.overlay.offsetX=0 thumbnails.overlay.offsetY=16

2. Overlay size: 131 (w) x 196 (H) See links for the actual overlay Overlay are placed in the resource folder for the current skin.

3. Run YAMJ

What is the expected output? What do you see instead?

What version/revision of YAMJ are you using? On what operating system?

What skin are you using?

What device are you using YAMJ on? Networked Media Tank, PlayonHD, Other?

Please provide any additional information below. Links to results and overlays http://kvisit.com/Ss7u7AQ (First run and overlays) http://kvisit.com/Szby7AQ (Second run, same settings)

I have tested with all variations of thumbnails.stretch and thumbnails.normalize.

I have both covers with originally correct aspect and incorrect aspects misplaced.

I started out trying this with XML overlays (thumbnails.xmlOverlay=true) but due to the misplacement I went over to try with (thumbnails.overlay=true) to minimise the complexity. I had the same problem with both settings.

Omertron commented 9 years ago

Comment #1 originally posted by Omertron on 2011-10-25T11:49:42.000Z:

Are there any possibilities that this one will be solved? For me it's a bit of a showstopper whether to release my skin/eskin publicly or not due to that I think a lot of people has set up the maxthreadsprocess > 1.

Omertron commented 9 years ago

Comment #2 originally posted by Omertron on 2011-10-25T12:23:19.000Z:

To speed things up, can you provide an example setup so that I can just apply it to a blank setup and replicate things?

Omertron commented 9 years ago

Comment #3 originally posted by Omertron on 2011-10-25T12:53:08.000Z:

You can find my skin in the dented boxes forums "DentedBoxes Forum / Eskins / Eskin Releases / evHell 2.0 beta" with some install notes.

Direct link to the overlay http://www.keepandshare.com/doc/3211256/xml-overlay-zip-october-20-2011-11-38-pm-736k?da=y

Direct link to the eskin files http://www.keepandshare.com/doc/3211255/eskin-zip-october-20-2011-11-38-pm-10-0-meg?da=y

Omertron commented 9 years ago

Comment #4 originally posted by Omertron on 2011-10-25T19:36:08.000Z:

I really need a simple setup that I can test with. I have no idea what are right and wrong results for what you are seeing.

Omertron commented 9 years ago

Comment #5 originally posted by Omertron on 2011-10-25T20:01:59.000Z:

If you look at the last post in the

Omertron commented 9 years ago

Comment #6 originally posted by Omertron on 2011-10-25T20:03:01.000Z:

The pictures are misplaced and it seems arbitrary and not the same from run to run.

Omertron commented 9 years ago

Comment #7 originally posted by Omertron on 2011-10-25T20:54:55.000Z:

This issue was updated by revision r2725.

Potential fix for misplaced overlay

Omertron commented 9 years ago

Comment #8 originally posted by Omertron on 2011-10-25T20:55:28.000Z:

Please test and report back

Omertron commented 9 years ago

Comment #9 originally posted by Omertron on 2011-10-25T21:18:09.000Z:

First run with 8 movies looks great. I will do a full run a bit later and let you know. Thanks a lot. If it is a success I'll publish the skin this weekend.

Omertron commented 9 years ago

Comment #10 originally posted by Omertron on 2011-10-26T05:05:59.000Z:

I an confirm that this issue has been resolved with 2725. Thanks again.

Omertron commented 9 years ago

Comment #11 originally posted by Omertron on 2011-10-26T05:58:03.000Z:

<empty>