[Ohrrpgce] SVN: james/8566 Add two new NPC movement types, "Chase You (Direct)" and "Avoid You (Dir

[Ralph Versteegen]

On 22 March 2017 at 06:44, James Paige <Bob at hamsterrepublic.com> wrote:

> Here is some thinking-out-loud:
>
> What if there was an NPC option to change the target of Chase/Avoid ?
>
> Possible values:
>
> * leader (current default)
> * hero slot 1,2,3
> * last hero in caterpillar
> * other NPC
>
> What would be the best way to implement NPC following?
>
> The easiest way would be to add a couple values to the NPC definition, one
> for NPC ID, and another for copy count. That would also leave open the
> option later on to add a convenient target-picker in the map editor in NPC
> placement mode... but all copies of an NPC ID would all need to have the
> same target.
>
> Perhaps a better option would be to make Chase/Avoid target an NPC
> instance property. That would require me to create a convenient
> target-picker in the map editor right now... but then I will need to
> implement npcloc-#.reld map lumps before I can implement it.... so not
> really any downside, just more upfront work I need to do to make it happen
> this way.
>

Being able to set the target is intereseting, not something I'd thought of!

Making it per-instance instead of per-definition definitely sounds like a
good idea, because in-game, it would definitely be better if it were
per-instance. (That's true for several other things that are currently
per-definition)

There's no reason you can't just extend the existing .L lump and replace it
with a .reld later; you wouldn't even need to resize any records. It sure
will be great to have that lump replaced though.

I was thinking about putting all data for a single map into a single reload
doc. And if part of it is missing (eg zonemaps, tilemaps) a separate lump
is loaded, so we can move over to the combined file very gradually, or even
keep maps 'unpacked' while editing them, for convenience of saving only
part. It would allow exporting a map as a single file.

I don't see why we shouldn't just use NPC references instead of ID and copy
count. Either way, if the NPC is deleted in the editor, it'll be broken.
And the only difference occurs when it's broken and then a new NPC with
that ref/copy count is created

Also, since you're adding XYPair utility functions, I'd like to point out
that you could have written in npcmove_direct_chase:
DIM dist as XYPair = t2 - t1
or even:
dist = (XY(catx(0), caty(0)) - XY(npci.x, npci.y)) / 20
(OK, operator/ was missing, so I just added it). In future I would prefer
to have a XYPair .pos instead of loose .x and .y members in UDTs.



> On Tue, Mar 21, 2017 at 10:27 AM, <subversion at hamsterrepublic.com> wrote:
>
>> james
>> 2017-03-21 10:27:31 -0700 (Tue, 21 Mar 2017)
>> 185
>> Add two new NPC movement types, "Chase You (Direct)" and "Avoid You
>> (Direct)"
>> The existing Chase/Avoid types have been recaptioned "Chase You
>> (Meandering)" and "Avoid You (Meandering)"
>> ---
>> U   wip/customsubs.rbas
>> U   wip/game.bas
>> U   wip/game.bi
>> U   wip/testgame/npcwalktest.rpg
>> U   wip/udts.bi
>> U   wip/whatsnew.txt
>> U   wip/yetmore2.bas
>> _______________________________________________
>> Ohrrpgce mailing list
>> ohrrpgce at lists.motherhamster.org
>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>
>
>
> _______________________________________________
> Ohrrpgce mailing list
> ohrrpgce at lists.motherhamster.org
> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.motherhamster.org/pipermail/ohrrpgce-motherhamster.org/attachments/20170322/befa8783/attachment.htm>