fixed NaN, but still not working