still not doing what I want it to