search

immersivecognition / unity-experiment-framework

UXF - Framework for creating human behaviour experiments in Unity
https://immersivecognition.github.io/unity-experiment-framework/
MIT License
214 stars 41 forks source link

Added recordData variable #102

Closed Avdbergnmf closed 2 years ago

Avdbergnmf commented 2 years ago

I needed something to disable data recording during some of the trials (familiarization trials). This seemed like the easiest way to accomplish that and figured others might be able to use it.

jackbrookes commented 2 years ago

Thanks! I will review when I can.

But generally I am not convinced - why would you ever want to discard data even for familiarisation # trials? Is storage space an issue? You could mark them with a setting, then filter them out in data analysis.

From: Jack Brookes @.> Sent: Friday, February 4, 2022 3:12:58 PM To: immersivecognition/unity-experiment-framework @.> Cc: Subscribed @.***> Subject: Re: [immersivecognition/unity-experiment-framework] Added recordData variable (PR #102)

Thanks! I will review when I can.

But generally I am not convinced - why would you ever want to discard data even for familiarisation # trials? Is storage space an issue? You could mark them with a setting, then filter them out in data analysis.

— Reply to this email directly, view it on GitHubhttps://github.com/immersivecognition/unity-experiment-framework/pull/102#issuecomment-1030077426, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFUG7XEGCZMKFFV5LEYOGSTUZPUHVANCNFSM5NSBBR5A. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you are subscribed to this thread.Message ID: @.***>

Avdbergnmf commented 2 years ago

Our experiment has a 5 min familiarization phase among some other stages (a training stage) that over ~30 participants will add up to a lot of additional data. It's not going to be a huge problem, but a problem nonetheless. I like to clean up my data as early as possible.

jackbrookes commented 2 years ago

@Avdbergnmf what does the output look like when e.g. saving to csv (trial_results)? Are there blank rows?

Avdbergnmf commented 2 years ago

With this code, no rows get added to the results, and no CSV files are created for the trackers for these trials. (Since it simply skips all the data related lines on the creation and cleanup of a trial.)

jackbrookes commented 2 years ago

I added this feature but with a different method. The property is called saveData and can be assigned to a whole session, block or trial. Thanks!