Menu

#5865 SCI: PQ1SCI: Stuck in briefing room

SCI: Police Quest 1
closed-fixed
3
2016-02-08
2011-10-08
No

When the briefing is about to start, pathfinding seems to get in the way and Sonny doesn't find his seat after being told to sit down (clashes with one of the people entering the room, the other guy resolves and sonny stops or the other way round)
Control is disabled at this point so there's no way to manually sit down.
I was able to get around this by loading and retrying and it works as it only happens half the time
Self compiled linux git master e5889af

Discussion

  • Tarek Soliman

    Tarek Soliman - 2011-10-08

    SAVEGAME: Wait and you will be told to sit down and won't be able to

     
  • Tarek Soliman

    Tarek Soliman - 2011-10-08
    • summary: SCI: PQ1: Stuck in briefing room --> SCI: PQ1SCI: Stuck in briefing room
     
  • digitall

    digitall - 2011-10-09

    I can confirm replication of this with the attached savegame and latest Git master.
    It doesn't seem to be linked to whether you load from launcher or in game.

     
  • digitall

    digitall - 2011-10-09

    With ./scummvm -d 5 --debugflags=Pathfinding, the following information is given:
    kRestoreGame(,4)
    Savegame in file pq1sci.004 ok, id 4
    [avoidpath] Pathfinding input:
    Start point: (193, 146)
    End point: (135, 134)
    Optimization level: 1
    Polygons:
    3: (72, 122) (190, 152) (202, 152) (255, 98) (146, 81) (67, 116) (72, 122);
    2: (158, 100) (176, 89) (212, 95) (195, 109) (158, 100);
    2: (100, 123) (136, 100) (219, 119) (191, 147) (144, 134) (145, 126) (134, 132) (100, 123);

    Returning path:
    (193, 146) (191, 147) (135, 134);

    With the bug not occuring...

     
  • digitall

    digitall - 2011-10-09

    Do difference in this output in the bug occuring case...

     
  • Tarek Soliman

    Tarek Soliman - 2011-10-09

    > No difference in this output in the bug occuring case...

    Indeed. I tried it both ways. I suspect it might be related to clicking to make the popup "go to your desk" go away.

     
  • digitall

    digitall - 2011-10-12

    Ah, worked it out. The bug here is that you don't have control to manually sit down after the dialog appears indicating that you should sit down.

    If you continue standing, Sgt Dooley enters and orders you to sit down three times before you get thrown off the force i.e. game over, so mouse control to "Use" the lefthand chair _SHOULD_ be possible in this scene.

    Prior to the dialog appearing, you can use the mouse to walk Sonny around and use the left chair to have him sit down. The pathfinding lockup only occurs if you don't move Sonny from the initial position he occupies in the room where he fouls with the NPC paths. If you move him, the bug does not occur... and he will automatically sit down after the dialog succesfully.

    It will take a SCI dev to look through the script disasm for this scene and see if this is a minor script issue with incorrectly disabling control after the dialog?

    tsoliman: It would help if you could run your copy of PQ1 in DOSBox and attach a savegame from there at the same point for behaviour comparison. Thanks.

     
  • Tarek Soliman

    Tarek Soliman - 2011-10-13

    SAVEGAME: same thing in dosbox

     
  • Tarek Soliman

    Tarek Soliman - 2011-10-13

    I was able to reproduce this bug in dosbox. Dosbox savegame attached.

     
  • Tarek Soliman

    Tarek Soliman - 2011-10-13
    • labels: --> 415215
     
  • digitall

    digitall - 2011-10-13

    Ah, so this is a script bug that occurs in the original. I will reduce the priority to indicate that this should be fixed if possible, but is not as critical.

     
  • digitall

    digitall - 2011-10-13
    • labels: 415215 --> Also occurs in the original
    • priority: 5 --> 3
     
  • Filippos Karapetis

    • assigned_to: nobody --> thebluegr
    • status: open --> closed-postponed
     
  • M. Kiewitz

    M. Kiewitz - 2015-04-17

    I just tried to trigger this bug and it doesn't happen - at least for me - using latest build.

     
  • M. Kiewitz

    M. Kiewitz - 2015-04-17
    • status: closed-postponed --> closed-later
     
  • M. Kiewitz

    M. Kiewitz - 2016-02-08
    • labels: Also occurs in the original --> Also occurs in the original, Script bugs
    • status: closed-later --> closed-fixed
    • assigned_to: Filippos Karapetis --> M. Kiewitz
     
  • M. Kiewitz

    M. Kiewitz - 2016-02-08

    Finally fixed by script patch in commit 4baa9963eeb9edb65c2b85c5d29f79027fd5607b

     
MongoDB Logo MongoDB