[Freeciv-Dev] Re: (PR#8624) New clima function to best handle terrain place, used to place poles.

["Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=8624 >

Marcelo Burda wrote:

> that is exactly i not want. this only find simplest singularities but
> not the central simetries in quincuntial. if iterator is well defined
> (as is in quincuntial patch!) this near_singularities find it!
> but evidently is_singular_map_pos is realy best in any other topologies.

I don't understand how the iterator method will find the singularity. 
The current iterators will happily duplicate points if the wrapping 
works out, and changing this seems hard.

Also, in a quincuncial topology you know exactly where the singularities 
are ((map.xsize/2,0) and so on).  So can't you easily check for them in 
is_singular_map_pos?

However is_singular_map_pos doesn't work well in hex topologies.  It is 
possible but not pretty.  So this may not be the best solution.

jason