<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 27 March 2018 at 13:26, James Paige <span dir="ltr"><<a href="mailto:Bob@hamsterrepublic.com" target="_blank">Bob@hamsterrepublic.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br><br>On Monday, March 26, 2018, Ralph Versteegen <<a href="mailto:teeemcee@gmail.com" target="_blank">teeemcee@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>I'm a bit concerned by:<br><br>+   IF rank = 0 ANDALSO gam.hero_pathing(rank).by_user THEN<br>+    'Only the leader activates NPCs, and only when doing built-in user pathing<br><br></div>Other commands for scripted hero movement don't have any affect on touch-NPC activation. And I assume pathing a touch-activated NPC past the hero doesn't prevent its activation either. Why should hero pathfinding be exceptional? If people don't want touch activation to occur, they could explicitly suspend that activation instead*.<br></div><div>And anyway, you only disabled the touch-NPC-activation check in update_hero_pathfinding, but didn't disable the check in the NPC movement code.<br></div><div><br></div></div></blockquote><div><br></div></span><div>No need for concern, actually :) That code you quoted there actually has nothing at all to do with touch NPC activation. That is "use" NPC activation when using mouse/touch controls.</div><div><br></div><div>It happens to strongly resemble touch activation, but is a separate thing. I can add a comment there to clarify that.</div></blockquote><div><br></div><div>Ah, whoops!<br></div><div>Right, so clicking on an NPC activates it, but the scripted equivalent is to write<br></div><div>  hero chases npc(me, npcref, true)<br></div><div>  wait for hero<br></div><div>  # Should maybe test if the NPC was reached and still exists<br></div><div>  use npc(npcref)<br></div><div>A builtin script to do all of that might be useful.<br></div><div>One thing worth considering is the ability to emulate builtin controls with script commands. For example suppose you have a minimap on the screen and want clicking on it to work the same as clicking on the map. Unfortunately there are all these exceptions in which scripted pathing differs from user-caused pathing. Maybe extra options to set by_user = YES would be useful. Not that important though.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">* In fact, there is no command to suspend NPC activation, except for "suspend obstruction" and "set npc usable"! The lack of a "suspend npc activation" command is a pretty huge oversight.<br></div></blockquote><div><br></div></span><div>That is something I ought to add. Thanks for the reminder :)</div><div class="HOEnZb"><div class="h5"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_extra"><br><div class="gmail_quote">On 27 March 2018 at 11:15,  <span dir="ltr"><<a href="mailto:subversion@hamsterrepublic.com" target="_blank">subversion@hamsterrepublic.co<wbr>m</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">james<br>
2018-03-26 15:15:12 -0700 (Mon, 26 Mar 2018)<br>
217<br>
Add a .by_user member to the HeroPathing object and use it to distinguish between built-in user pathing and scripted hero pathing<br>
Options to cancel pathing on textbox, menus, and battles only apply when .by_user = YES<br>
---<br>
U   wip/game.bas<br>
U   wip/<a href="http://game.bi" rel="noreferrer" target="_blank">game.bi</a><br>
U   wip/<a href="http://game_udts.bi" rel="noreferrer" target="_blank">game_udts.bi</a><br>
______________________________<wbr>_________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org" target="_blank">ohrrpgce@lists.motherhamster.o<wbr>rg</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.org<wbr>/listinfo.cgi/ohrrpgce-motherh<wbr>amster.org</a><br>
</blockquote></div><br></div>
</blockquote>
</div></div><br>______________________________<wbr>_________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org">ohrrpgce@lists.motherhamster.<wbr>org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.<wbr>org/listinfo.cgi/ohrrpgce-<wbr>motherhamster.org</a><br>
<br></blockquote></div><br></div></div>