This should be done in constructor