updates after pr