Great stuff indeed ! I'm definitely gonna use it (and modify it).
Like Forien, I'll add my 2 cents : 3. Rivers, moutains, coasts, ridges....Cities are not build "just there", there are reasons they were built there.
I would second the rivers options. While cities on major trade routes did happen, they were/are often more regularly associated with sources of water (rivers or springs). So something more than just "coast" would be great.
