And yet again this is becoming another famous APDAF design. Start on the major things, such as deck layout, the general shape and look of things. Did you read the comments and suggestions or look at the illustration? I think not. Here is what you have done:
~added 2 more lifeboats
~added railings, scuppers, anchors, a central island, and more windows
What still needs doing:
~change funnels to a more merchant looking shape
~deboxify the bridge
~that forward crane really should be moved back and another put in front of it
~remove windows from the superstructure (almost all of them need to go)
~fix the weird curve you have on the stern island, it makes no sense on a vessel of this type
Now, look at what has been done, and what needs to be done. The second list contains things more important than those on the first. Actually do something to make this work. Once again, look in the Merchant Ships thread in real designs, what you are looking for amy not be on the first page, but then keep going throught until you find some ships from this era.
I have looked at the illustration.
The reason that the bridge is boxy is that it is pre-fabricated. (Think of modern shipbuilding)
The curve on the stern is a mess up.