search

carrien / free-speech

Analysis and plotting code for speech neuroimaging experiments.
MIT License
4 stars 5 forks source link

Include current time (HH:MM:SS) as item stored in expt structure, get_gender->get_height #28

Closed Comeani closed 3 years ago

Comeani commented 3 years ago

I can't remember if this was talked about when we initially added date to the default expt file, but the situation with the trial number mismatch in the iEEG data collection have me thinking that it might be a good idea hold on to time of expt creation alongside date. I realize that 'time' could be a confusing field name alongside 'timing', so we could just do something like setting date to datetime('now') instead. Also made a quick swap of get_gender to get_height as default behavior for asking for participant gender.

cwnaber commented 3 years ago

Updating to start using get_height seems good.

I'm ambivalent on if we have a 'date' and 'time' field, or just update the 'date' field to use datetime. I think I slightly prefer the way you've coded it here.