netstim / leaddbs

Lead-DBS is a MATLAB toolbox facilitating deep brain stimulation electrode localization and connectomic neuroimaging.
https://www.lead-dbs.org
Other
90 stars 98 forks source link

[BUG]: Naming file crashes Lead Connectome process #1111

Open diegoadriperez opened 9 months ago

diegoadriperez commented 9 months ago

Describe the bug The bug is a mismatch between a file name that is expected versus the file name that is actually saved.

To Reproduce Steps to reproduce the behavior: Running the lead connectome on these settings:

If you stop the code once it starts using ea_map_coords.m at line 103. Continue through until you get to the SPM normmethod (line 188). It should find a file named: "b02anat_t2-spm.mat" but instead finds a file named "b02anat_t2spm". The "-" versus the "" crashes the code.

Expected behavior It should find the file and continue as normal with no issue.

Screenshots N/A

Environment:

Additional context Add any other context about the problem here.

ningfei commented 9 months ago

sorry, Lead Connectome is not yet ready for 3.0 with the BIDS folder structure. We plan to release a new version in the beginning of next year.

diegoadriperez commented 9 months ago

Should I downgrade to the version before for the time being?

ningfei commented 9 months ago

For LeadConnectome, you can use the classic version. Otherwise better use v3.0 version. You can also setup your LeadDBS from github. So you can switch between the two versions easily. For more info, can check out our manual: https://netstim.gitbook.io/leaddbs/installation