Kind of hard to tell on the yard, but this is what I see in whats shown:
Sidings are to short, you will only be able to fit 1 or 2 cars cleanly on the shortest of them.
I don't see a run-around or escape track. This is a must if you don't want captured engines waiting on cars to be pulled to release them.
You would actually be farther ahead to make a smaller yard (fewer tracks) and increase your length. Save the unused turnouts for another location where they will actually serve a purpose.
As a suggestion, your yard should support as many cars as is required to make up as many trains as you plan to run. That is, if you plan to run 3 trains with 10 cars each, your yard should hold at least 30 cars. Ideally a yard is full when at 50% capacity, so a 60 car yard would be even better.
Something else to consider is sorting and staging tracks, that is an inbound, an outbound, and track leads to break, sort, store. While you are at it, a RIP track is a good idea. This gives you someplace to park stock that isn't quite up to standards so it can be easily located for repairs. And last, if you are running in a caboose era, you will need a lead for caboose storage that has immediate access to both the inbound and outbound leads.
Confused yet?, hehe.
Check out this site for a better understanding of yards and how they are designed.
http://www.housatonicrr.com/yard_des.html