scanner-research / esper-tv

Esper instance for TV news analysis
Apache License 2.0
39 stars 10 forks source link

likely useful to add minutesToSeconds and secondsToMinutes conversions #87

Closed kayvonf closed 6 years ago

kayvonf commented 6 years ago

I see a number of "* 60" and "/ 60" 's in the code. To avoid potential typos/errors and to improve clarity let's make some helper routines.

def minutesToSeconds(x) def secondsToMinutes(x)

Alternatively, we could use datetime.timedelta.

jhong93 commented 6 years ago

This is added in now along with a change to use timedeltas.