Optional event_col in AAF and CPH fitters

@spacecowboy, this PR adds the option for the `event_col` parameter (default None). If left as `None`, then we assume all non-censored (similar to the kmf fitter). 

Also, `duration_col` is now a mandatory argument. 
Or "assume all events" ?