redundant columns in session.trials

[alexpiet]

@nicain There is redundant information in the session.trials table

session.trials.go is True if it a go true
session.trials.catch is True if it is a catch trial (exact opposite of session.trials.go)
session.trials.sham_change is True if it is a catch trial (exactly redundant of session.trials.catch)

I can think of why you might want to have both 'go' and 'catch' as one-hot encoding, but I can't think of a reason why you might want both 'catch' and 'sham_change.'

I am using the LimsApi on the internal branch of the SDK

[nicain]

@dougollerenshaw can you confirm this? If so, I will remove sham_change as a column.

[dougollerenshaw]

Confirmed. Column adds no information.

…

On Jun 24, 2019, at 1:38 PM, nicain ***@***.***> wrote: @dougollerenshaw can you confirm this? If so, I will remove sham_change as a column. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.