change to list comprehension