I appreciate the feedback; I’m new to mapmaking and have been researching so much to make my things look proper. I was focused on nobility structures recently and hadn’t gotten too far into seige prevention.

The main keep is in the central most section; the knight garrison and a small citadel are inside there. I need better imposing assets for the keeps. They would be #1 and #5 respectively.

Yes! This is a capital city; I tried to design it so a food source existed inside the walls for longer sieges.

This city lies in a valley on a coast on the border of a jungle; it sits on a slope down toward the water. I do usually lift up the lighthouse, and should have used a rock or something there.