started adding factorial