netstim / leaddbs

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

[BUG]: Stimulation amplitude cannot be set correctly (old programmer) #1210

Open alessiaca opened 4 months ago

alessiaca commented 4 months ago

When trying to set the stimulation amplitude in the old programmer the GUI behaves in an unexpected way. When I input a number (e.g. 3), it is not immediately updated but takes ~2 seconds. Also, if I input a number with a decimal point, it shows NaN. The only way I found to set numbers with decimal points was to first input the number, wait 2 seconds, click outside and inside of the field again and then add the decimal point. I think it may be related to the Matlab version because I did not have this problem with Matlab 2022.

See the video https://drive.google.com/file/d/1J1e2ymktn0_Krx0CgrIEC4u6lYZpAgxj/view?usp=sharing

ningfei commented 3 months ago

Hi Alessia, thanks for reporting this and sorry for the late reply! You were right. Seems supporting for GUIDE based GUI is not working so well for MATLAB version >= 2023a. I saw the same issue when I tested on Linux and Mac using MATLAB 2024a. But it works with MATLAB 2022b. Will see if I can find a solution for this.

ningfei commented 2 months ago

Hi @alessiaca it should work better with this latest commit 53490dbf. But might be slower since it uses AppDesigner now. Let me know if you still have issue using it.