Add new date format and better logic to test multiple date formats

AllenNeuralDynamics / aind-subject-nwb

MIT License

0 stars 0 forks source link

Add new date format and better logic to test multiple date formats #9

Closed alejoe91 closed 3 months ago

alejoe91 commented 4 months ago

@Ahad-Allen this replaces #8

I made a fresh PR with no conflicts and added a better logic for date formats management.

Can you test on your side?

Ahad-Allen commented 3 months ago

Testing now

Ahad-Allen commented 3 months ago

I found a bug that occurred when there was a mismatch of some data times having timezones associated with them but the session start time not having one. I added a fix where we assume pacific time zone. This may not be ideal, would like to hear your thoughts on it