scanForRocks stopped working

For test results, bug reports, announcements of new builds etc.

Moderators: winston, another_commander, Getafix

Post Reply
User avatar
Eric Walch
Slightly Grand Rear Admiral
Slightly Grand Rear Admiral
Posts: 5536
Joined: Sat Jun 16, 2007 3:48 pm
Location: Netherlands

scanForRocks stopped working

Post by Eric Walch »

The command scanForRocks is broken. Today I tried to test my mining mission under 1.70, but the miners just flew past the asteroids. I replaced it for the new "scanForNearestShipWithAnyPrimaryRole: asteroid boulder" and things worked again. So it must have been the scanForRocks that is faulty. I didn't test if it does finds boulders but it does not find asteroids.

But an other reason I tested this mission is because it always generated the unscoopable cargo (In my case splinters). It still does. I can't target it so I can not meke the extended target dump but only an complete entety dump. This was the result of the culprit:

<ShipEntity 0x28644600>{"Splinter" ID: 453 position: (-17284.7, 13758, 742602) scanClass: CLASS_CARGO status: STATUS_BEING_SCOOPED} mass 1264.86 <AI 0x1820d170>{"nullAI.plist" in state: "GLOBAL" for Splinter 453}
User avatar
Frame
---- E L I T E ----
---- E L I T E ----
Posts: 1477
Joined: Fri Mar 30, 2007 8:32 am
Location: Witchspace

Post by Frame »

which means the Rock Hermit Locator also do not work in 1.7

havent gotten around to test it...
Bounty Scanner
Number 935
User avatar
JensAyton
Grand Admiral Emeritus
Grand Admiral Emeritus
Posts: 6657
Joined: Sat Apr 02, 2005 2:43 pm
Location: Sweden
Contact:

Post by JensAyton »

It appears that the code is supposed to be looking for boulders (equivalent to scanForNearestShipHavingRole: boulder twice, instead of looking for boulders first and then falling back to asteroids.

This is now fixed, although looking at it I’m not sure it whether it ought to be doing an any-role check or a primary-role check. Then again, if this isn’t a candidate for an any-role check, I’m not sure what is. :-)
Post Reply