search

makebrainwaves / BrainWaves

An easy-to-use platform for EEG experimentation in the classroom
MIT License
60 stars 26 forks source link

Display behavioral results #79

Closed Yury-Shevchenko closed 5 years ago

Yury-Shevchenko commented 5 years ago

The main functionality for the display of behavioral results was added (with plotly.js and react-plotly.js). "Papaparse" is used to extract data from csv files, "simple-statistics" - for the analysis of data. The behavioral results (reaction time and accuracy) are displayed at the "Analysis" stage of the app. For reaction time, the means and standard errors of each subject dataset are shown next to each other (using histograms). For accuracy, the number of correct responses is shown for each subject dataset (also with histograms). Users can switch between different datasets and dependent variables, can download and save figures.

teonbrooks commented 5 years ago

@Yury-Shevchenko, this is looking good. I see that this includes LSL in the pr. do you want to rebase to the master branch? if you want to include the lsl part here, we would like to put the stream initiation under a pref and have some checkbox in the flow to enable or disable it with the experiment selection.

Yury-Shevchenko commented 5 years ago

@teonbrooks I've just realised that the new update with behavioral results also included the previous update with lsl. It would be probably better to separate these two things into separate brunches. I suggest I will update the "display-behavioral-results" brunch so that it will only have the display code, and will create a separate brunch with "lsl" with the next task to make a checkbox to enable/disable lsl.

Yury-Shevchenko commented 5 years ago

I have added a button to save selected datasets as one csv file. In this file, there are two additional columns - "subject" and "session".

Yury-Shevchenko commented 5 years ago

In the last version ("display and aggregate"), data for each subject are displayed in whisker plots (with data points alongside) next to each other across conditions (colors still have to me adjusted), the scales for the response time and accuracy are fixed. Pressing the export button saves the aggregated data (time and accuracy). The checkbox "Remove outliers" can be activated to get rid of responses that above or below 2 SD from the mean.

sdikker commented 5 years ago

Awesome! 👍🏼👍🏼 see you all monday for the weekly meeting.

Suzanne Dikker +1-212-920-4348 Sent from mobile

On Jul 5, 2019, at 3:34 PM, Yury-Shevchenko notifications@github.com wrote:

In the last version ("display and aggregate"), individual data are displayed next to each other across conditions (colors still have to me adjusted), the scales for the response time and accuracy are fixed. Pressing the export button saves the aggregated data (time and accuracy). The checkbox "Remove outliers" can be activated to get rid of responses that above or below 2 SD from the mean.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

Yury-Shevchenko commented 5 years ago

In the last commit "display aggregated data" the following things are done: 1) The csv files with aggregated data also can be displayed inside the app. The files should be positioned inside of the project folder and include "aggregated" in the name. 2) There is a checkbox button to show individual data points on the figure. The individual data points are shown next to the box plots, since they share styling now (if one is transparent, the other one is also transparent). I will work on making only box plots transparent (when the individual points are shown), if you need that. 3) The area for the selected datasets is extended to accommodate more selected files. Unfortunately, scrolling inside ui.multiple.dropdown has an issue with the dropdown menu, which is moved under the list of selected items and looks ugly. I will continue to search for a fix for this problem.

Yury-Shevchenko commented 5 years ago

If you want to merge this pull request with the project, someone should with write access should do it. Apparently, I do not have write access here.

teonbrooks commented 5 years ago

thanks @Yury-Shevchenko! I will see if I can give your permissions

sdikker commented 5 years ago

On Mon, Jul 15, 2019 at 3:50 PM Yury-Shevchenko notifications@github.com wrote:

In the last commit "display aggregated data" the following things are done:

  1. The csv files with aggregated data also can be displayed inside the app. The files should be positioned inside of the project folder and include "aggregated" in the name.

amazing!

  1. There is a checkbox button to show individual data points on the figure.

    perfect.

  2. The area for the selected datasets is extended to accommodate more selected files. Unfortunately, scrolling inside ui.multiple.dropdown has an issue with the dropdown menu, which is moved under the list of selected items and looks ugly. I will continue to search for a fix for this problem.

sounds good, thank you!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/makebrainwaves/BrainWaves/pull/79?email_source=notifications&email_token=AH3DAVGMZVVY6VVRDQ75MMTP7R6BTA5CNFSM4HZRWAC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ5YBMQ#issuecomment-511410354, or mute the thread https://github.com/notifications/unsubscribe-auth/AH3DAVDERP6KNZ2TCC3B3N3P7R6BTANCNFSM4HZRWACQ .

-- Suzanne Dikker suzannedikker.net +1.212.920.4348 (US mobile) +31.6.2835.2199 (Amsterdam mobile)