fixed passing parmasd in constructor