this should be done in init