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

Prasad #12

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

Prasad #12

wants to merge 37 commits into from

Conversation

pshirvalkar
Copy link
Collaborator

I only want to request a pull of the file makeDataBaseRCSdata.m, and not the other files. I could not figure out how to do this in pull request, but i have dragged the file here.

FYI - Using Roee's earlier code 'getDatabaseRCS' i created a new function that extracts all relevant data using the open mind code, and is very fast (does not use deserializeJSON).

Right now adaptiveSettings are not included as part of the database, but can be added.
See brief help at top for use.

makeDataBaseRCSdata.m

pshirvalkar added 3 commits January 30, 2021 18:39
@roeegilron
Copy link
Member

Hi Prasad - nice job!
Looks like this fails on our dat: dbout has 7 fields, whereas fieldorderhas 14... line 162 is the error. Also, will line 121 work for all groups? The code should be general / work in all use cases - e.g. for groups using 1 > more programs.

@kristinksellers
Copy link
Collaborator

image

Running Matlab 2018b

@kristinksellers
Copy link
Collaborator

@pshirvalkar Please add the version of findFilesBVQX that you'd like used to the repo

@kristinksellers
Copy link
Collaborator

kristinksellers commented Feb 21, 2021

Line 184 -- We updated the processing pipeline to now create AllDataTable.mat (output of ProcessRCS.m) --> master branch. See DEMO_LoadRCS.m for an example of how to interact with this file (also summarized below -- the steps below occur after loading the file):

dataStreams = {timeDomainData, AccelData, PowerData, FFTData, AdaptiveData}; % Desired data streams to include
[combinedDataTable] = createCombinedTable(dataStreams,unifiedDerivedTimes,metaData);

significant updates to RCSpainpreprocess.m, RCSlogs.m, and RCSplotterscript.m

Also edited Roee's basic spectrogram script in RCS plotter to not fill white gaps for small amounts of missing data.
-make database updated to use the old database if present and add to it
-updated plotter script and RCSPlotter to fix some errors with plotting / inputs
- pain score import now for RCSpain
big updates to the RCS plotter, which now can be used to analyze closed loop.

amended plotter methods  to be able to plot subsessisons with multiple sampling rates.
NOT fully working yet, will push shortly to update
…aseRCS and dependents

Also started working on combining left and right though not complete
created new M file to compile all patients, and used parallel for loop for text logs
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

Successfully merging this pull request may close these issues.

3 participants