[fixed] Path information for mixed movement group

Progress reports and musings from the developers on the current gaming tools.

Moderators: dorpond, trevor, Azhrei

Post Reply
Lee
Dragon
Posts: 958
Joined: Wed Oct 19, 2011 2:07 am

[fixed] Path information for mixed movement group

Post by Lee »

Fixed bugs reported in this topic

Moving forward, "[fixed]" (on my posts) will refer to an issue fixed, but with the patch for it following at a later time as the file also includes other unrelated changes which I have not documented and fully tested yet; in other words, this post is a placeholder/notice for now, so that other devs will know what's been already done.

Using the supplied campaign file found on the post above, the group is made up of tokens that are either detached or snapped to the grid. The following are screenshots for proof-of-fix:
Movement led by unsnapped token
Image
Results
Image
Movement with way points
Image
Results
Image
Movement led by snapped token
Image
Results
Image
Movement with multiple steps
Image
Results
Image
Lastly, Wiki: getMoveCount() now returns a value of 0 for newly dropped tokens on the map. If there are any questions, do feel free to ask.
Last edited by Lee on Tue Oct 30, 2012 6:11 pm, edited 2 times in total.

tiorthan
Cave Troll
Posts: 84
Joined: Fri Aug 24, 2012 8:56 am
Location: Germany
Contact:

Re: [patched] Path information for mixed movement group

Post by tiorthan »


Lee
Dragon
Posts: 958
Joined: Wed Oct 19, 2011 2:07 am

Re: [patched] Path information for mixed movement group

Post by Lee »

:lol: yes, I think it is. I hopped in here to grab the link to put it on that post to let you know that I was in that code area and will look into it :)

User avatar
jfrazierjr
Deity
Posts: 5176
Joined: Tue Sep 11, 2007 7:31 pm

Re: [patched] Path information for mixed movement group

Post by jfrazierjr »

Lee wrote:...stuff..
Replied to Private Message, hopefully with something helpful.
I save all my Campaign Files to DropBox. Not only can I access a campaign file from pretty much any OS that will run Maptool(Win,OSX, linux), but each file is versioned, so if something goes crazy wild, I can always roll back to a previous version of the same file.

Get your Dropbox 2GB via my referral link, and as a bonus, I get an extra 250 MB of space. Even if you don't don't use my link, I still enthusiastically recommend Dropbox..

Lee
Dragon
Posts: 958
Joined: Wed Oct 19, 2011 2:07 am

Re: [fixed] Path information for mixed movement group

Post by Lee »

Thanks JFJ. It was.

Can I get this thread moved to the developer notes section? I think it being here without a patch to test makes it the wrong place for now. Changing the OP and title to reflect [fixed] instead of [patched]

Lee
Dragon
Posts: 958
Joined: Wed Oct 19, 2011 2:07 am

Re: [fixed] Path information for mixed movement group

Post by Lee »

@aliasMask Continuing on the fix illustrated on the first post, I was able to return precision as well have handling code for each mix context (e.g snapped key token w/ unsnapped followers). The problem with it was that I had to crossover the threshold and make changes to some critical model classes such as ones for Token and Path. An example was to incorporate an origin point for path computation. One would think that it wouldn't be needed, but it turns out that the token's coordinates get updated during the drag, losing valuable reference points along the way.

Since I mentioned that I was releasing 2 builds with all the fixes (1 for each UPnP candidate library), this is the last one I'll be applying before building the releases. So the question is, which is the preference here?:

a. Apply the simple fix mentioned on the 1st post and release the more complex path fix in my fork after 1.3 final.
b. Apply the "complex" fix now, knowing that it might not make it into 1.3 final. Due to it's nature, it'll take more time to do this.

On a side note, the problem with FoW and getMoveCount mentioned in related discussions have also been handled. Let me know what's okay with you. I'm partial to a., if only to have something out right away.

Thanks.

User avatar
aliasmask
RPTools Team
Posts: 9023
Joined: Tue Nov 10, 2009 6:11 pm
Location: Bay Area

Re: [fixed] Path information for mixed movement group

Post by aliasmask »

At this point we've lived with it the way it is for awhile, so the simple fix to get things out sooner is preferable. The path update has a work around as well and it is also in relation to a semi-fringe case.

Lee
Dragon
Posts: 958
Joined: Wed Oct 19, 2011 2:07 am

Re: [fixed] Path information for mixed movement group

Post by Lee »

Alas, I spoke too soon. After extracting the fix from my fork, even the simpler fix made a change to the Token model. I'll detail what this means for people who don't know on the patch post.

Post Reply

Return to “Developer Notes”