bugFreeciv - Bugs: bug #25067, Paradrop should honour...

 
 
Show feedback again

bug #25067: Paradrop should honour pythagorean_diagonal?

Submitted by:  Jacob Nevins <jtn>
Submitted on:  Sat Sep 17 13:35:28 2016  
 
Category: NoneSeverity: 3 - Normal
Priority: 5 - NormalStatus: None
Assigned to: NoneOpen/Closed: Open
Release: Operating System: Any
Planned Release: 3.0.0Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sat Sep 17 13:43:34 2016, comment #1:

Thinking about this a bit more, the obvious thing would be to calculate real polar distance, yielding a circular range on all topologies. But pythagorean_diagonal has no effect on hex topologies (as there are no diagonal moves); currently the paradrop range is hexagon-shaped there; so it would be inconsistent to make it circular.

Defining the paradrop range be the set of tiles you could get to with the equivalent number of movement points (approximately) and no terrain costs would be consistent. That would give a square for rectangular maps, a circle for square+pythagorean games, and a hexagon for all hex maps. This can be implemented by checking for (!hex && pythagorean_diagonal) in paradrop range calculations.

Jacob Nevins <jtn>
Project Administrator
Sat Sep 17 13:35:28 2016, original submission:

We probably can't practically have every single map distance calculation honour the pythagorean_diagonal mode added in patch #4819. But paradropping is obviously movement related, so not having it honour this mode creates an obvious exploit in a ruleset with both paratroops and pythagorean_diagonal.

In principle this applies to 2.6 and trunk, but fixing it might be too fiddly to do at this stage as we stabilise S2_6 (especially given the different implementation of paradrop on trunk). Not sure.

Jacob Nevins <jtn>
Project Administrator

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jtn (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Error: not logged in

     

     

    No Changes Have Been Made to This Item
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup