Opened 6 years ago

Closed 8 months ago

#6410 closed defect (fixed)

SCI: QFG1VGA - pathfinding bug in Yorick's room

Reported by: misterhands Owned by: sluicebox
Priority: normal Component: Engine: SCI
Keywords: pathfinding original has-pull-request Cc: sluicebox
Game: Quest for Glory 1

Description

Latest Daily Win64
QFG1 VGA

While testing the fix for bug 3585793 (well done M. Kiewitz btw!) I found another path-finding bug that hangs the game.
This bug looks similar to bugs 3585189:
https://sourceforge.net/tracker/?func=detail&aid=3585189&group_id=37116&atid=418820
and 3604939:
https://sourceforge.net/tracker/?func=detail&aid=3604939&group_id=37116&atid=418820

To trigger it, in Yorick's room click at the bottom right corner of the door on the right.
Please see the pic in the zip archive, I've added a red box on it to be perfectly clear.

Ticket imported from: #3615032. Ticket imported from: bugs/6410.

Attachments (1)

Pathfinding bug.zip (226.4 KB ) - added by misterhands 6 years ago.
This is the same save as the one in bug 3585793, so disregard the "bug! push that button" as that bug is now fixed.

Download all attachments as: .zip

Change History (7)

by misterhands, 6 years ago

Attachment: Pathfinding bug.zip added

This is the same save as the one in bug 3585793, so disregard the "bug! push that button" as that bug is now fixed.

comment:1 by wjp, 6 years ago

I can confirm this; I've seen it a number of times while experimenting in that room.

comment:2 by csnover, 2 years ago

Keywords: script removed

comment:3 by dafioram, 2 years ago

Keywords: pathfinding added

comment:4 by sluicebox, 8 months ago

Cc: sluicebox added
Keywords: original has-pull-request added

Script patch for this: https://github.com/scummvm/scummvm/pull/1519

This is a bug that also occurs in the original, our pathfinding algorithm is doing the same thing's Sierra's did in this case. Sierra rewrote all of this door's code in the Mac version and changed all the relevant positions and obstacle sizes, so the bug only occurs in PC versions.

The easiest way to test this is to just warp to room 96. If you already have a saved game in that room, this patch will take effect immediately, it's not one the ones where you have to reload the room first. (unless you're already locked up)

comment:5 by Filippos Karapetis <bluegr@…>, 8 months ago

In 4e1963d:

SCI: Fix QFG1VGA Yorick door lockup, bug #6410

comment:6 by bluegr, 8 months ago

Owner: set to sluicebox
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.