would be nice to have variable names also follow pep8 by making this ```ids```