Difference between revisions of "SCI/TODO"

From ScummVM :: Wiki
< SCI
Jump to navigation Jump to search
(pq3 vm issue got workarounded)
(Remove outdated SCI32 TODOs/bugs)
 
(51 intermediate revisions by 6 users not shown)
Line 1: Line 1:
{{Infobox_TODO|
taskname=SCI Engine TODO|
techcontact=[[SCI]] Engine Team|
subsystem=Engine|
}}
= Missing features =
= Missing features =


== SCI0-SCI1.1 missing features ==
== SCI0-SCI1.1 missing features ==
=== Graphics ===


=== Sound ===
=== Sound ===
* Voice mapping. This is used by all SCI games to assign hardware voices to midi channels in the AdLib and FB-01 drivers. So, for example, if a song tries to play more than 9 notes at the same time, we need to stop an earlier note. We currently do this round-robin, while we should be respecting voice assignments. If a game sets 3 voices for piano, we should not take one away unless 4 piano notes are played, then it's OK to stop one piano note. Currently if a song were to use 5 notes at once we would just play them all, even if that midi channel was set to use 2 voices, and those 3 extra notes might stop more important notes, e.g. in the melody. This has been implemented for SCI0, but it's not hooked up to AdLib at the moment. Also we're not sure if the initial voice assignments are in the MIDI stream in SCI1 (they're not in SCI0). Of course, channel remapping has an effect on this too
* Voice mapping. This is used by all SCI games to assign hardware voices to midi channels in the AdLib and FB-01 drivers. So, for example, if a song tries to play more than 9 notes at the same time, we need to stop an earlier note. We currently do this round-robin, while we should be respecting voice assignments. If a game sets 3 voices for piano, we should not take one away unless 4 piano notes are played, then it's OK to stop one piano note. Currently if a song were to use 5 notes at once we would just play them all, even if that midi channel was set to use 2 voices, and those 3 extra notes might stop more important notes, e.g. in the melody. This has been implemented for SCI0, but it's not hooked up to AdLib at the moment. Also we're not sure if the initial voice assignments are in the MIDI stream in SCI1 (they're not in SCI0). Of course, channel remapping has an effect on this too
* Improve Amiga (SCI01/1) and Mac sound support ([[SCI/Specifications/Sound/SCI0 Resource Format#Amiga Sound (SCI1)/Macintosh Sound (SCI1/1.1)|same format]])
* Improve Amiga (SCI01/1) and Mac sound support ([[SCI/Specifications/Sound/SCI0 Resource Format#Amiga Sound (SCI1)/Macintosh Sound (SCI1/1.1)|same format]])
* Proper song volume support, currently volume is reset to max whenever a new song is loaded
* kDoAudio subops 12 and 13 (used in Freddy Pharkas: subop 12 is called while people are talking and subop 13 is called while the narrator's voice is heard)
* kDoAudio subops 12 and 13 (used in Freddy Pharkas: subop 12 is called while people are talking and subop 13 is called while the narrator's voice is heard)
* Implement a CMS driver
* Adapt the CMS driver to support earlier SCI versions too.
 
* MT-32 is only using 10 channels (1-10), but our music code always assumes 16 channels leading to lost notes
=== Parser ===
* Rewrite the MIDI parser (which is using ticks) so that it uses byte offsets instead
* Support for non-English text input


=== Mac-specific ===
=== Mac-specific ===
* Support for the SCI1 Mac version of kGetEvent
* Support for the SCI1 Mac version of kGetEvent
* Support for the SCI1 Mac version of kNewWindow
* Support for the SCI1 Mac version of kNewWindow
* Support for modified SCI1.1+ scripts (all but KQ6)
** Looks like some parts are LE and some parts BE (as opposed to KQ6 having all BE)
* Fix QuickTime videos - assigned to clone2727
** The videos now play in KQ6 Mac, but require multiple edit list support (see [[Mohawk/TODO#Video|here]])
* Support for SCI1.1 partially hardcoded icon bar
* Support for SCI1.1 partially hardcoded icon bar
** Images stored in PICT resources (tags: 'IBIN' (not-selected), 'IBIS' (selected), 'PICT' (inventory image))
** Images stored in PICT resources (tags: 'IBIN' (not-selected), 'IBIS' (selected), 'PICT' (inventory image))
** kPlatform subop 4 is called various times for the icon bar. One call sets up the scripts for the icons, etc.
** kPlatform subop 4 is called various times for the icon bar. One call sets up the scripts for the icons, etc.


== SCI2-SCI3 missing features ==


== SCI2-SCI2.1 missing features ==
* Support for Macintosh versions of games
 
'''This list is VERY incomplete, SCI2-SCI2.1 support is still in very early stages'''
 
=== Graphics ===
* Proper support for text bitmaps
* Support for Robot (RBT) videos
* kRemapColors used for transparency
** Also used in QFG4 demo (SCI1.1), but mainly a SCI32 item
(TODO)
 
=== Sound ===
(TODO)
 
=== VM ===
* Support for "chunk" scripts
(TODO)
 
== SCI3 missing features ==
 
'''Support for SCI3 is non-existent'''
 
=== Graphics ===
* Support for Duck TrueMotion 1 videos (only used in Phantasmagoria 2)
(TODO)
 
=== Sound ===
(TODO)
 
=== VM ===
* Support for the new SCI3 scripts
 


= Known bugs =
= Known bugs =
Line 65: Line 33:


=== Graphics ===
=== Graphics ===
* KQ5 Mac: The game uses a screen size of 320x190 and the offTop needs to be set to 0. However, setting the offTop to 0, messed up the priority screen. Without changing offTop, the menu bar will flicker when hovering over it.
* KQ6/CD: several sync resources aren't available in the resource files. The windows version does only use the additional sort of "raw-data" appended to those sync resources. We are currently using the sync resources inside kPortrait instead of the raw data, that's why lip sync isn't working for those cases. We need to find out how this raw data is interpreted, probably we could even create virtual sync data out of it for the DOS scripts to use
* LSL5: waking up from the flute dream doesn't fully remove the dream cloud - issue is because scripts show a textbox and remove them after a kDrawPic call that redraws the plane picture, this results in picNotValid not to get set (which is correct), so the new picture isn't fully displayed on screen. This was fixed in multilingual lsl5 version
* LSL6: =NOT OUR BUG= when getting killed in the gym, view 19488 not found happens. The same happens in sierra sci. It should have been view 453. Similar issue happens when giving your room key to the guy in front of the locker and shower rooms. view 26478 is not found in that case (ffs. bug #3038331).


=== Sound ===
=== Sound ===
Line 76: Line 40:
* KQ5: End credits song is broken (resource 699). we are now filtering out invalid channels, but at the end of the song it plays back garbled somewhat. This video proves that it worked fully in original sci http://www.youtube.com/watch?v=GerEZ68YSzc - the resource is uncompressed inside resource.000 right at the end, this is definitely no decompression or resource size error
* KQ5: End credits song is broken (resource 699). we are now filtering out invalid channels, but at the end of the song it plays back garbled somewhat. This video proves that it worked fully in original sci http://www.youtube.com/watch?v=GerEZ68YSzc - the resource is uncompressed inside resource.000 right at the end, this is definitely no decompression or resource size error
* LSL6: when saving when music is fading out (and restoring) the music will stay on that level and keep playing - we should check for fading and modify selectors to the endpoint
* LSL6: when saving when music is fading out (and restoring) the music will stay on that level and keep playing - we should check for fading and modify selectors to the endpoint
* SQ1: it seems sierra sci wasn't able to pause samples at all. We are getting pause sample call when getting killed by acid drops and we pause the sample, but this isn't the way sierra sci behaved. In that case the sample continued. Maybe they just stopped it from looping again. LSL1 seems to pause samples on kDoSound(pause) calls although thats sci1early sound wise
=== SCI0-01 parser ===
* PQ2: when loading a savegame of room 66, the synonyms don't work. In particular, this breaks "examine tank".


=== VM ===
=== VM ===
* Castle of Dr. Brain: Saving and restoring in room 380 will result in a broken sentence puzzle (last puzzle in that room) - for testing replace 577504302400 with 5775042E2400 in script.380 - this will make it possible to use as many hint coins on the locked cupboard, so that one doesnt have to solve the hangman puzzle at all - this may be a regression since last playthrough (end of january 2010) - workaround: play through this room at once w/o restoring
* Island Dr. Brain/wii exclusive(?): when being in the first puzzle room, going to menu doesn't really work on Wii. The menu is visible for around 0.1 seconds and then disappears automatically - caused by MoveCursor not setting cursor - needs to get further investigated
* KQ5: Path-walking - in the town some people "jump" around sometimes, also happens to ego in rare cases as well (sq5 right at the start when walking down)
* LSL6: =NOT OUR BUG= sometimes the tram does not stop. This is a game script issue and not an engine one, but we should still figure out what goes wrong and fix it. To make it work -> see SCI bugs
* LSL6: =NOT OUR BUG= sometimes the tram does not stop. This is a game script issue and not an engine one, but we should still figure out what goes wrong and fix it. To make it work -> see SCI bugs
* PQ2: Room 26 isn't exitable - happens if you warp from inside the car at the start into room 26 - happened to me during the game although I'm not unable to reproduce it anymore. I guess I did something differently last time.
* PQ2: Room 26 isn't exitable - happens if you warp from inside the car at the start into room 26 - happened to me during the game although I'm not unable to reproduce it anymore. I guess I did something differently last time.
* QFG1 EGA: when buying something from the shop and the merchant is reading at that time, it will get bought twice
* QFG1 EGA: when buying something from the shop and the merchant is reading at that time, it will get bought twice
* QFG1 VGA: When winning any random monster fight, the game uses the unimplemented kMergePoly function. This means that the hero can walk over the bodies of slain enemies (though it's a purely cosmetic issue)
* QFG1 VGA: ending music isn't played at all. no sound calls are done, so I guess it's VM.
* QFG1 VGA: ending music isn't played at all. no sound calls are done, so I guess it's VM.
* SQ5: Engine Abort in Room 119 when Roger is using Scrub-o-matic to clean crest : "lookupSelector(): Attempt to send to non-object or invalid script. Address was 0000:0000!"
* SQ5: When beating WD40, sometimes screen doesn't scroll up and roger is stuck inside the log. May also be a script bug


=== Known Warnings ===
=== Known Warnings ===
* Iceman: dancing with girl WARNING: [VM] Error: Attempt to add 0 to pointer 001b:07ce, type 2: Pointer arithmetics of this type unsupported!
* KQ5: Warning - kStrAt with offset 0xFFFF is called when graham gets the staff, also happens when getting the heart in the forest
* KQ5: Warning - kStrAt with offset 0xFFFF is called when graham gets the staff, also happens when getting the heart in the forest
* KQ5: Warning - try to peek at 0:0, 0:2 and 0:4 most of the time when dialogue windows appear
* KQ5: Warning - try to peek at 0:0, 0:2 and 0:4 most of the time when dialogue windows appear
Line 101: Line 53:
* LSL6: WARNING: Attempt to read character from non-raw data! during end credits
* LSL6: WARNING: Attempt to read character from non-raw data! during end credits
* PQ2: WARNING: ADLIB: Invalid patch 124 requested!
* PQ2: WARNING: ADLIB: Invalid patch 124 requested!
* PQ3: During End Sequence, WARNING: kDoSound(setHold): Slot not found (0026:0fc4)! and WARNING: Unhandled SCI MIDI command 0x51 (parameter 1)!
* QFG1EGA: Invalid adlib patch 1xx used, occurs several times in the game
* QFG1EGA: Invalid adlib patch 1xx used, occurs several times in the game
* SQ4: During intro - "WARNING: Unhandled SCI MIDI command 0x0 (parameter 0)!"
* SQ4: During intro - "WARNING: Unhandled SCI MIDI command 0x0 (parameter 0)!"
* SQ4: On first screens in future Xenon, repeated - "WARNING: kDoSound(pause): Slot not found (0009:02c4)!"
== SCI2-SCI2.1 known bugs ==
'''This list is VERY incomplete, SCI2-SCI2.1 support is still in very early stages'''
* GK1: When leaving Madame Cazanoux's house, Gabriel will walk through the wall to get to the door. Madame Cazanoux walks around the door, however.
* GK1: Save/Restore menus do not work properly.
* GK1: Sometimes sprites are not erased from the screen.
* GK1: In ''very'' rare cases, the background picture is not updated.
** This happened to me twice: Moving from the Gedde tomb left and from the Schloss Ritter bedroom to the hallway.
== SCI3 known bugs ==


'''Support for SCI3 is non-existent'''
== SCI2-SCI3 known bugs ==


(TODO)
* Please check the [https://bugs.scummvm.org/report/12 bug tracker]

Latest revision as of 01:49, 24 April 2017

TODO List
Name SCI Engine TODO
Technical Contact(s) SCI Engine Team
Subsystem Engine

Missing features

SCI0-SCI1.1 missing features

Sound

  • Voice mapping. This is used by all SCI games to assign hardware voices to midi channels in the AdLib and FB-01 drivers. So, for example, if a song tries to play more than 9 notes at the same time, we need to stop an earlier note. We currently do this round-robin, while we should be respecting voice assignments. If a game sets 3 voices for piano, we should not take one away unless 4 piano notes are played, then it's OK to stop one piano note. Currently if a song were to use 5 notes at once we would just play them all, even if that midi channel was set to use 2 voices, and those 3 extra notes might stop more important notes, e.g. in the melody. This has been implemented for SCI0, but it's not hooked up to AdLib at the moment. Also we're not sure if the initial voice assignments are in the MIDI stream in SCI1 (they're not in SCI0). Of course, channel remapping has an effect on this too
  • Improve Amiga (SCI01/1) and Mac sound support (same format)
  • kDoAudio subops 12 and 13 (used in Freddy Pharkas: subop 12 is called while people are talking and subop 13 is called while the narrator's voice is heard)
  • Adapt the CMS driver to support earlier SCI versions too.
  • MT-32 is only using 10 channels (1-10), but our music code always assumes 16 channels leading to lost notes
  • Rewrite the MIDI parser (which is using ticks) so that it uses byte offsets instead

Mac-specific

  • Support for the SCI1 Mac version of kGetEvent
  • Support for the SCI1 Mac version of kNewWindow
  • Support for SCI1.1 partially hardcoded icon bar
    • Images stored in PICT resources (tags: 'IBIN' (not-selected), 'IBIS' (selected), 'PICT' (inventory image))
    • kPlatform subop 4 is called various times for the icon bar. One call sets up the scripts for the icons, etc.

SCI2-SCI3 missing features

  • Support for Macintosh versions of games

Known bugs

SCI0-SCI1.1 known bugs

Graphics

Sound

  • The Tandy/PCjr driver needs more work
  • Castle Dr. Brain: In the programming puzzle room 280 the music is using one bad instrument
  • Castle Dr. Brain: in the first room, some notes aren't played during the memory puzzle
  • KQ5: End credits song is broken (resource 699). we are now filtering out invalid channels, but at the end of the song it plays back garbled somewhat. This video proves that it worked fully in original sci http://www.youtube.com/watch?v=GerEZ68YSzc - the resource is uncompressed inside resource.000 right at the end, this is definitely no decompression or resource size error
  • LSL6: when saving when music is fading out (and restoring) the music will stay on that level and keep playing - we should check for fading and modify selectors to the endpoint

VM

  • LSL6: =NOT OUR BUG= sometimes the tram does not stop. This is a game script issue and not an engine one, but we should still figure out what goes wrong and fix it. To make it work -> see SCI bugs
  • PQ2: Room 26 isn't exitable - happens if you warp from inside the car at the start into room 26 - happened to me during the game although I'm not unable to reproduce it anymore. I guess I did something differently last time.
  • QFG1 EGA: when buying something from the shop and the merchant is reading at that time, it will get bought twice
  • QFG1 VGA: ending music isn't played at all. no sound calls are done, so I guess it's VM.

Known Warnings

  • KQ5: Warning - kStrAt with offset 0xFFFF is called when graham gets the staff, also happens when getting the heart in the forest
  • KQ5: Warning - try to peek at 0:0, 0:2 and 0:4 most of the time when dialogue windows appear
  • LSL5: Warning - Invalid character in kReadNumber input (happens right at the start of the game)
  • LSL6: WARNING: Attempt to read character from non-raw data! during end credits
  • PQ2: WARNING: ADLIB: Invalid patch 124 requested!
  • PQ3: During End Sequence, WARNING: kDoSound(setHold): Slot not found (0026:0fc4)! and WARNING: Unhandled SCI MIDI command 0x51 (parameter 1)!
  • QFG1EGA: Invalid adlib patch 1xx used, occurs several times in the game
  • SQ4: During intro - "WARNING: Unhandled SCI MIDI command 0x0 (parameter 0)!"

SCI2-SCI3 known bugs

Retrieved from "https://wiki.scummvm.org/index.php?title=SCI/TODO&oldid=23753"