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

zef_transfer_matrix broken by missing variable definition #147

Closed
SeSodesa opened this issue Oct 4, 2022 · 2 comments
Closed

zef_transfer_matrix broken by missing variable definition #147

SeSodesa opened this issue Oct 4, 2022 · 2 comments
Assignees

Comments

@SeSodesa
Copy link
Collaborator

SeSodesa commented Oct 4, 2022

When trying to run the EEG routine, the following error is thrown in the CPU branch of zef_transfer_matrix:

>> zef_eeg_lead_field
Unrecognized function or variable 'n_parallel'.

Error in zef_transfer_matrix (line 243)
            parpool(n_parallel);

Error in zef_lead_field_eeg_fem (line 207)
[T, Schur_complement, ~] = zef_transfer_matrix(...

Error in zef_lead_field_matrix (line 158)
    [zef.L, zef.source_positions, zef.source_directions] = zef_lead_field_eeg_fem( ...

Error in zef_eeg_lead_field (line 13)
zef = zef_lead_field_matrix(zef);

I'm not sure where n_parallel is supposed to come from, but this is a blocker for the stable release. Looks like commit 47adc81 is the culprit behind this issue.

Note: Once this is fixed, please write the hash of the fixing commit here, so that the issue may be marked as resolved by that commit. Even better would be to create a branch for the fix and then merge it into main_development_branch once the fix is complete.

@sampsapursiainen
Copy link
Owner

Corrected.

@SeSodesa
Copy link
Collaborator Author

SeSodesa commented Oct 4, 2022

Seems to be fixed by c8f7a1b.

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

No branches or pull requests

2 participants