Difference between revisions of "SAGA/TODO"

From ScummVM :: Wiki
Jump to navigation Jump to search
(Remove duplicates, update on getActorFrameRange warnings)
(Opcodes)
Line 19: Line 19:
** Cutaway related: Anim::returnFromCutAway
** Cutaway related: Anim::returnFromCutAway
** Video and cutaway related: Scene::restoreScene
** Video and cutaway related: Scene::restoreScene
* Investigate what the following opcodes are:
** sf87
** sf88
** sf89
** sf103 (is_actorfacing?)
* Investigate if opcodes sf64, sf69, sf70, sf73 and sf92 are really dummy (null) opcodes
* Conversation options are not shown correctly (wrong colors). Also, if a conversation option is selected, the character does not say it and we immediately jump to the other character's reply instead
* Conversation options are not shown correctly (wrong colors). Also, if a conversation option is selected, the character does not say it and we immediately jump to the other character's reply instead
* Actor walking speed is not correct when exiting from rooms
* Actor walking speed is not correct when exiting from rooms

Revision as of 20:15, 6 June 2007

Status

  • ITE is completable with rare minor gfx glitches.
  • IHNM shows complete intro. Also, some chapters are partially working, but it's not possible to return to the character selection screen once a chapter is completed (or abruptly stopped by a wrong player option) and ScummVM exits instead.
    • Chapter 1 (Gorrister): Completable with full points but has some glitches
    • Chapter 2 (Ellen): Not fully tested
    • Chapter 3 (Benny): Not fully tested
    • Chapter 4 (Nimdok): Not completable, it's impossible to use the scalpel with the doctor in the surgery room
    • Chapter 5 (Ted): Not completable, it's impossible to interact with the icon in the chapel or the books in the room of the witch
    • Chapter 6 (Endgame): Not possible to test yet


IHNM ToDos

General

  • [Important] Implement sfScriptGotoScene differences for IHNM, which will allow to go back to the character selection screen once a chapter is complete
  • Implement several missing functions:
    • Psychic profile related: sfPsychicProfile and sfPsychicProfileOff
    • Video related: Anim::startVideo, Anim::returnFromVideo, Anim::endVideo
    • Cutaway related: Anim::returnFromCutAway
    • Video and cutaway related: Scene::restoreScene
  • Investigate what the following opcodes are:
    • sf87
    • sf88
    • sf89
    • sf103 (is_actorfacing?)
  • Investigate if opcodes sf64, sf69, sf70, sf73 and sf92 are really dummy (null) opcodes
  • Conversation options are not shown correctly (wrong colors). Also, if a conversation option is selected, the character does not say it and we immediately jump to the other character's reply instead
  • Actor walking speed is not correct when exiting from rooms
  • After chapter selection AM has intro speech. After the speech, the actor is incorrectly shown for a brief moment. In the original, the screen colors get garbled momentarily, so this seems to be an issue with the scripts in the original game, albeit it affects ScummVM in a different manner
  • After the introduction, during the character selection screen, 5 warnings are thrown (Frame List ID = 0 for actor index 0,1,2,3,9). This happens at other places, too
  • It seems that depth image isn't correct. Different format?
  • Options panel shows up without textures on the buttons and some options are not functioning yet (e.g. the load/save dialogs)
  • Voices volume isn't implemented yet
  • Implement saving and loading
  • The way that verbs are assigned internally is ugly and needs to be cleaned up
  • Characters can get stuck in environmental objects (might be because of mistakes in actor animation). E.g. Gorrister can get stuck in the corridor railings
  • Only numbers 1-4 on the keyboard are working to select a converse option, the rest are only selectable with the mouse
  • When the spiritual barometer changes, the character's portrait changes to "happy" or "sad", but the background color is updated after moving to another screen
  • Actor::getActorFrameRange keeps throwing wrong frametypes 0x0 and 0x2 for actorId 0x2000 and 0x2001 during character selection and when Ted enters the castle

Chapter 1: Gorrister

  • When flushing the toilet, the door is moved to a weird spot (looks like a line above the toilet stall)
  • The hatch inside the zeppelin is not drawn correctly
  • When trying to climb up the stairs to the second floor of the zeppelin with Gorrister, the game tries to disable the hitzone of the tear at the end of the corridor, however the hitzone returned is wrong. Currently, the hitzone is not disabled using a hack in the code, which makes the staircase work correctly in that room, thus preventing a crash.
  • Some in-game animations are not shown in ScummVM (look at the general TODOs section)
    • in the screen just outside the tear, where Gorrister's heart is, when trying to pick up the heart with the rope placed, there should be an animation where Gorrister is picking up his heart, which is not shown in ScummVM
    • when using the knife on an air bag, there should be an animation where the bag is losing gas
    • the zeppelin landing animation is not shown
    • the zeppelin takeoff animation is not shown
    • the animation of Gorrister killing Max in the honky tonk is not shown
  • sfPlaceActor throws a "Wrong frameOffset 0x1" warning when using the knife to scare the mice away in the kitchen
  • Using the jukebox in the honky-tonk changes "use" to "use with". Perhaps an issue with whichObject()
  • Looking at the cages in the zeppelin's engine room can cause ScummVM to crash
  • If you look outside the porthole in the room where the gun is (Gorrister's chapter, exit starting room, go left and enter the right door), the screen will change, showing the ship's engine. However, after the animation is complete, the scene doesn't update properly to show the room again, and it shows the last frame of the engine animation instead. This occurs because of the unimplemented parts of Anim::returnFromCutaway(), in particular the missing restoreScene() function
  • When doing a good deed which raises the spiritual barometer with Gorrister (e.g. when Gorrister is wiping his hands on the tablecloth), the music should change to a more "light" theme. This is not happening in ScummVM

Chapter 2: Ellen

  • Ellen's walking animation is not showing at all
  • When Ellen is selected at the character selection screen and while AM is talking to her, she should have a different expression in her portrait, which changes once she gets zapped outside the pyramid. Her portrait is static in ScummVM

Chapter 3: Benny

  • Pathfinding algorithm bugs:
    • When walking with Benny from the forest to the caves, the actor can't find the path properly.
    • Also, when clicking on a cave entrance with Ben, he starts walking to the cave and then moves a step back, thus he can't enter it. If the user clicks on the lower left of each cave (or other spots around the cave entrance), Ben enters the cave correctly

Chapter 4: Nimdok

  • [Important] It's not possible to use the scalpel with the doctor in the surgery room. This makes it impossible to complete Nimdok's chapter. Perhaps a bug with the pathfinding algorithm - the doctor seems to be in a "no go" area
  • When Nimdok enters the hospital at the beginning of the chapter and talks with the other doctor, the text of their dialog goes off screen
  • Some in-game animations are not shown in ScummVM (look at the general TODOs section)
    • The animation of the golem killing Nimdok is not shown

Chapter 5: Ted

  • [Important] It's not possible to interact with the icon in the chapel (e.g look at it, take it). This makes it impossible to complete Ted's chapter
  • [Important] When trying to interact with the books in the room of the witch, Ted turns around and moves out of the room. This makes it impossible to complete Ted's chapter. Perhaps a bug with the pathfinding algorithm - the books seem to be in a "no go" area
  • When a wrong monitor is selected with Ted, normally the door closes, screen goes black and AM says "Oh oh oh sorry Ted, try again", and we're taken to the character selection screen again. ScummVM just exits, because this case is currently not handled by sfScriptGotoScene yet

Chapter 6: Endgame

[Not possible to test yet]

Original engine bugs

  • When loading games, the music played is the one that was playing from the scene where the game was loaded. For example, if a saved game is loaded from the character selection screen, which was made on chapter X, then once the game is loaded, the music from the character selection screen is still played


ITE ToDos

  • Amiga versions of the game would be nice to support. Though Amiga porter which once appeared at our forums doesn't reply to sev's letters. Maybe another dev should try that?
  • Original DOS demo. It is based on very early version of engine, so many structures are different. I've mapped most of resources but they stay untested.
    • Move actors and objects list to our code. I've found them but not yet processed.

Requests

  • We need to find out where following functions are called:
    • SF_cycleColors -- all implementation bits sit in palanim.cpp
    • sfPlayVoice -- used in ITE CD
  • There are currently two versions of ITE that are not uniquely identified: the Wyrmkeep "Win32 CD Version" and the original "DOS/Win32 CD Version". When adding games, the user gets to do the disambiguation, but what happens with the detection when playing the game?

BUGS

  • Occasional graphics glitches in ISO rooms. (At least in the Rats maze.) Most notable when entering rounded doors. (3d draw sequence is wrong)
  • pressing ESC during making fire - breaks anim sequence

Interface

  • wrong Option button draw sequence - some dots have inproper color.
  • handleConverseUpdate scrolls converse too quickly.
  • handleCommandUpdate scrolls inventory too quickly.

Original engine bugs

  • Actors are drawn (some pixels above status bar) at Chota's falling. - Original engine draws its too - so it is not our bug


Long-term ToDos

Bugs/misfeatures

  • Make actor animation Event-driven. See FIXME in actor.cpp. Maybe it should be INTERVAL_EVENT
  • Inspect Font module - free memory, GameFontIds <-> FONT_ID
  • In many places RSC_LoadResource() is called but memory is not freed afterwards, i.e. there is a huge leak
  • Optimize screen output. Now we blit whole screen for each frame.
  • When using the original XMIDI music resources, some songs play at the wrong speed. The worst example I've heard of this is the "Elk Fanfare" during the intro. It could be the meta events we don't handle, but at least the tempo event doesn't seem to make any difference. Maybe something in the data header instead...?
  • Remove use of floating point, especially doubles.