With that many buildings running right up to the wall, this city looks like it's experiencing some crowding. I'd expect to see more structures crowded around the gates.

There should also probably be some wide empty spaces for shipyards or dry docks. A port this size would need to have the capability to at least repair vessels, if not build them entire.

How high are the cliffs? You have some major streets running perpendicular to them, and if they're very tall at all, those streets are very steep.

I like the topography of this city, and the forts on the cliffs' edges are a nice touch. I can see where this city might have been a hotly contested site--maybe with rival nations holding opposite banks of the river and building their castles in an attempt to control the shipping going upriver.