Still not functional. Need to handles zeroes