ack, string_to_datetime handles being passed a datetime but should probably change these back for performance