these should really be vals, what was I thinking