openmw.org

This doesn't solve the case of what to do when there are no nav points. In this case, I think something like "Pick a point in the wander distance at random and go there" is the solution. I think the correct result is to walk nowhere. There's at least one NPC in vanilla that's supposed to ...

@scrawl @TorbinC @zini Thinking about this a bit more, AiWander has 3 scenarios when trying to figure out a destination for an actor to wander to. There are 2 or more nav points within the wander distance. There is 1 nav point within the wander distance. There are no nav points within the wander dis...

I've put together a very simple routine for AiWander to handle the “no path grid points” case. Basically, it picks a point thats “Wander Distance” from the NPC's initial location, and heads for it, doing castRay() to check if the destination is accessible. https://github.com/dteviot/openmw/tree/AiWa...

I'm creating this so we can have a permanent, easily findable, record of my screw-up. Original problem was bug https://bugs.openmw.org/issues/1317. where Erene Llenim did not wander because his wander distance was not big enough to include 2 waypoints. In Vanilla, Erine walks between the two waypoin...

Studing Morrowind.esm, Tribunal.esm & Bloodmoon.esm, there are 5 "swim only" creatures. "Dreugh", which has an AiWander of 1000 units. "Slaughterfish", which has an AiWander of 1000 units. "Old Blue Fin" (slaughterfish_hr_sfavd), which has an AiWander of 5...

Noticed a problem introduced by this commit. In "Balmora, South Wall Cornerclub" Phane Rielle always faces the wall, instead of towards the room. I do not believe there is a bug. "phane rielle" has an AI of wander 64 with a speed of 46. Looking at the Path Grid, this means there...

DataVisualizationHtml.zip 02.CheckEachFrameForReadingWaypoint.png I did a bit some work to visualize the "walking in circles" problem. I used the following code to record data for each frame in AiCombat: if (actor.getCellRef().getRefId() == "frinnius posuceius") { ESM::Position ...

@scrawl Can we fix this for all AiPackages at once please? Actually, I've updated AiPackage.pathTo() to call the buildSyncedPath(), which I believe fixes it for all packages but AiWander (which doesn't need the synced path) and AiCombat, where I explicitly call buildSyncedPath(). So, done. My longer...

scrawl wrote:The first two commits look ok, yes.

Done. https://github.com/OpenMW/openmw/pull/611
Thanks.

@scrawl PR #606 https://github.com/OpenMW/openmw/pull/606 contains three commits. https://github.com/dteviot/openmw/commit/66420f9d81398206bb094efe51745d1d2c0927b1 removes duplicated code from AiTravel. https://github.com/dteviot/openmw/commit/08cae020e93effac43c90b5cea59acbb30d59678 fixes "run...