The default fortress radius should definitely not be larger! Just raise the fortress radius in rmgen/misc.js (iberWallRadius line 193) and load a few maps. There's just not enough space on most. However, I can easily add an additional keyword argument to make the radius adjustable for each map (though I doubt this will be largely used).
I dug out the old code for more organic looking fortresses and adjusted it a bit. I got the size more stable at the same randomness. Added it and it can be setup in line 203 of rmgen/misc.js. They are closed by a linear wall so not all parts of the wall are bended. I'll try to reduce the length of the linear part further.
I replaced the entries with gates just to show how it looks. I will change them back before committing ofc. To make them playable just replace 'gate' with 'entry' in rmgen/wall_builder.js line 634 (for irregular polygonal fortresses) and for the others in rmgen/misc.js line 203, 206, 209 and 212.
Since there are 4 shapes working fine (for me) perhaps the style could be chosen randomly... I'd like that.
iber_wall_civ_bonus2012-5-4b.zip 135.95K 14 downloads
Edit: Since it's already in the SVN repo I assume you like it
It seams you're not a big friend of discussions
However, that was not considered final for me . There is much debug logging left and if we only want to use one fortress shape the other shape's code (documented out anyways) should be removed from misk.js. IMO. I'll post a cleaner version (only the files to update) later somewhere...
Edited by FeXoR, 05 May 2012 - 06:51 AM.