Open nguyen-td opened 5 months ago
Yes, these functions already accept a leadfield matrix plus an atlas (Brainstorm). Did you try the call you proposed.
Yes but there are some error messages, for example due to this line
that prevents arrays from passing through. There are some other error messages after fixing that (adding 'float' to the list) too. But I can work on it then.
Right now, we need a pointer to the head model and source model to run source reconstruction using
pop_roi_activity
. But technically, we only need a lead field, right?Can we modify the the
pop_roi_activity
androi_activity
functions in a way that makes them accept the raw lead field matrix/array instead of requiring pointers? Say we have a pre-computed lead field with the dimension (30 x 5003 x 3), where 30 is the number of channels, 5003 the number of voxels and 3 the orientation and pass this array directly to the function, then we could skip almost all steps and jump right to this line:https://github.com/sccn/roiconnect/blob/e80177fda427af5fb9ba656787e5f3b304b8f804/roi_activity.m#L250
For example, you could download a standard lead field here (New York head) and just pass the lead field. So instead of having to run everything here (e.g.
pipeline_connectivity
)we could simply run something like this (of course, you would need to match the EEG channels):
I think this would make the code more flexible. What do you think @arnodelorme?