#8310 closed patch
sword2 cutscenes
| Reported by: | SF/roever | Owned by: | SF/ender |
|---|---|---|---|
| Priority: | normal | Component: | Engine: Sword2 |
| Version: | Keywords: | ||
| Cc: | Game: | Broken Sword 2 |
Description
I've done some work on the Broken Sword 2 Animations.
The current code works as following: the animations are compressed to mp2 elementary streams. Additionally I've generated palette files that assign 256 color paletted to images sequances. These palettes were taken from the original animations. The player does now take the output of the mp2lib and reduces these to a palette index using a lookup table. This is relatively fast and gives satisfying results except for areas that are extremely comic like. Here we get artefacts.
The lookup table for the color reduction process is calculated while playing.
The appended patch includes the player. You still have to modify the config.mak to include the path to the mpeg2.h include and the lib. The 2 new files I added are extra.
The patch will make scummvm play the intro as long as the speech is going. The music of the animation is missing for the music to work we need more features in the mixer. We need to find out how far a sample has been played. I could not find the necessary functions.
I've prepared all animations. All together they are about 34 Mb. If someone wants to test the patch he only needs the intro files which are about 10 Mb.
All right I hope I've written enough to make you decide on wether you want to go this way or not.
Andreas
Ticket imported from: #874510. Ticket imported from: patches/415.
Attachments (2)
Change History (15)
by , 21 years ago
| Attachment: | mpg2anims.tar.gz added |
|---|
comment:1 by , 21 years ago
Update: The patch now also plays the background music but this is not really synchronized with the video. The mixer misses some information for that.
I also fixed a few bugs and removed old code.
The intro plays well now.
comment:3 by , 21 years ago
| Owner: | set to |
|---|
comment:4 by , 21 years ago
can this be closed now?
also I think the videos need to re-encoding due to all the mpeg artifacts in the current ones
comment:5 by , 21 years ago
Re-encoding wont help much as long as we stay with the current scheme (mpeg 2 compression + palettization) The mpeg2 will always introduce some artfacts around the lines the animation have in the cartoon parts of the images. These will get highlighted because of inaccuracies in the color selection. I had to make some speed <-> quality decisions that ecspecially show in the cartoon parts.
comment:6 by , 21 years ago
how much of a speed difference does it make to have higher quality video?
comment:7 by , 21 years ago
Quite a bit as each frame will have to be re-dithered.
As I've said before, just use overlays on >8bit systems to avoid having to do the conversion in the first place. Full quality, no speed impact.
comment:8 by , 21 years ago
My idea (hope) was that the differences introduced by mpeg are not too big. Then I took the palettes that the original animation used and hoped that when I take the color value returned from mpeg and find the best fittint palette entry that that is the same as in the smacker animation.
mpeg returns 3 values per pixel (actually 6 bytes per 4 pixel but that doesn't matter here) that would lead t a lookup of 256*256*256 bytes. calculateing suche a big lookup in between the frames takes too long, so I quantisized the values by dividing by 8.
You should be able to change that if you change the SHIFT value in animation.h. The lower the value, the less the color distortion but the more time is required to calculate the lookup table. A value one less required 8 times as much memory and time for the calculation. The parameter has not influence on the display speed.
As Ender said. A 32 bit overlay would be best for our purposes as the current scheme is quite expensive because the frame gets copied serveral times with color conversion.
comment:9 by , 21 years ago
The current code seems to have problems playing back a cutscene with a 640 x 480 resolution, check bug #884568.
comment:10 by , 21 years ago
Oops can not append a file in the buglist, so I put it here. It's for bug 884568.
comment:11 by , 21 years ago
| Status: | new → closed |
|---|
comment:13 by , 6 years ago
| Component: | → Engine: Sword2 |
|---|---|
| Game: | → Broken Sword 2 |
mpeg2 player files version 2