forgot to initialize values