As to the forest, it looks like the edges haven't been given enough love in order to stop the repeating so go back and try to squeeze in an extra tree along the edges. The pattern is technically seamless in that the trees all connect but there it too much white-space along the edges and that makes the repeat noticeable, extra trees or lines will kill some white-space and help out. Another option would be to cut that white space down so that you don't have to have fat trees along the edges that create the white-space. For example if the base image of the pattern is 100 pixels cut it down to like 97 and then re-align the trees along the edges and then reset it as a pattern.

Mountains look pretty good to me but maybe some more wiggliness to the edge-lines will make them less rigid and more up and down flowing - also some wiggliness in the hatching too.

All in all, not too shabby so keep pluggin away at it. Making patterns that tile seamlessly is probably one of the hardest things to do by hand and the fact that you're even attempting is a sign of good things. Once you do get it right, make sure you post a copy of it here so that we can all download it instead of doing our own We'd be really grateful and send you fruit baskets...they make a lovely gift.