TypeError: descriptor '__sub__' requires a 'datetime.datetime' object but received a 'str'

[soliverc]

In line 16, the user specified end date was not converted to a datetime object. It was throwing an error. I fixed it manually by adding datetime.strptime(recency_end_date, '%m/%d/%y') around recency_end_date.

From the rfm_utils.py file:

Before:

def convert_transaction_to_user(df, recency_end_date=None):
    id_col = df.columns[0]
    date_col = df.columns[1]
    transaction_col = df.columns[2]
    df[date_col] = pd.to_datetime(df[date_col])

    if recency_end_date is not None:
        end_date = recency_end_date
    elif recency_end_date is None:
        end_date = datetime.today()

After

def convert_transaction_to_user(df, recency_end_date=None):
    id_col = df.columns[0]
    date_col = df.columns[1]
    transaction_col = df.columns[2]
    df[date_col] = pd.to_datetime(df[date_col])

    if recency_end_date is not None:
        end_date = **datetime.strptime(recency_end_date, '%m/%d/%y')**
    elif recency_end_date is None:
        end_date = datetime.today()

[soliverc]

By the way I have no idea how to do a pull request. I attempted it but can't seem to figure it out. Gonna make another attempt.