Opened 6 years ago

Closed 3 weeks 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)

Changed 6 years ago by misterhands

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 Changed 6 years ago by wjp

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

comment:2 Changed 19 months ago by csnover

Keywords: script removed

comment:3 Changed 18 months ago by dafioram

Keywords: pathfinding added

comment:4 Changed 3 weeks ago by sluicebox

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 Changed 3 weeks ago by Filippos Karapetis <bluegr@…>

In 4e1963d:

SCI: Fix QFG1VGA Yorick door lockup, bug #6410

comment:6 Changed 3 weeks ago by bluegr

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