made changes according to class discussion, still not working