When I looked at the site I was impressed. I really like what you've done and visually it really jumps out at you. That is a stunning image and the rest of the design borrows from exceedingly well. Only things I noticed was that the links need to more obviously links, at the minute they just look like normal text. A bottom border or a different colour would help separate them. Also on some of the pages, the bullet point are doing strange things. I'm not sure why.

Now I really like this site and as I always do when critiquing sites I did a quick 'view page source' and what I saw upset me a little. First step put your css is an external style sheet. Then do the same with your JS. The final and most important step is to change to a tableless layout. I know its a big job but tables where never meant for layout and you could cut down on you code a lot if you did this. Its just better to work with a tableless layout. One reason why its important to do this is because seeing as this a new site, you probably won't redevelop it for a little while and with that design I see no reason why you should. However, this means that over the next few years as more site's are built to W3C standards yours will be left behind. I would seriously look into rejigging it and removing the tables.

You also need to look at removing presentation from the html and really separating style from content as well as using semantic code. Lists for link etc. Its not easy but its the right way to do things.

Great design, just improve that code and you'll have a winner.

Pete.