search

jyn514 / GradeForge

Courses available from my.sc.edu
GNU General Public License v3.0
1 stars 0 forks source link

Use strptime instead of rolling our own code #49

Closed jyn514 closed 6 years ago

jyn514 commented 6 years ago

As mentioned here, dateutil can handle 24 hour time for us, instead of the current mess in army_time. Just need to look up the API.

jyn514 commented 6 years ago

Looking at this a second time, dateutil looks like overkill, we probably want strptime instead. We'll have to play around with re.sub to make the formatting consistent, so army_time will have to stay.

jyn514 commented 6 years ago

No I take it back, we want dateutil so we can accept 24-hour time as input. It's bundled with pandas so it shouldn't be a problem.