Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

date_created field in header is ambiguous #129

Closed
galenlynch opened this issue Apr 26, 2017 · 0 comments
Closed

date_created field in header is ambiguous #129

galenlynch opened this issue Apr 26, 2017 · 0 comments

Comments

@galenlynch
Copy link
Contributor

Hi,

The date_created field in the .continuous file header has a format which is both hard to parse and is ambiguous. The date format seems to be "d-u-yyyy hmms" where d is the day of the month, u is the three letter abbreviation for the month, yyyy is the year, h is one or two digit hour, mm is the two digit minute, and s is the one or two digit second. Times can therefore be four to six digits long, and many five digit times can have two interpretations. For example the time "11111" could be parsed as either 11:11:01 or 01:11:11.

galenlynch added a commit to galenlynch/plugin-GUI that referenced this issue Apr 26, 2017
As discussed in open-ephys#129, the date_created field of .continuous file headers does
not have fixed width days of the month, hours, or seconds, which makes parsing
the header difficult and often ambiguous. I have changed the RecordNode class
to produce fixed width datestrings, which should fix this problem.

I was unable to compile with these changes, but the previous commit (226bfab)
won't compile for me either. However, I can make these changes to an earlier
commit and compile successfully.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant