ok, now work on update position
make clear seperation