MaxvandenBoom / matmef

Matlab wrapper for MEF library
GNU General Public License v3.0
1 stars 3 forks source link

Matlab crashes when loading data using a relative path on Mac/Linux #6

Open jezdip1 opened 1 year ago

jezdip1 commented 1 year ago

hello, I am trying to load some sample data from https://github.com/msel-source/sampledata using various Matlab versions (2018, 2021, 2022) and various gcc/g++ versions (10,11) and Matlab always crashes. any idea how to make your codes work? My configuration xubuntu 22.04 Matlab 2018, 2021a, 2022a gcc 10,11 (tested) 10 is recommended by mathworks as the compatible compiler

just simply cloned your repo and made the MEXes with successful outputs having some warnings, but only when mexing the write_mef_segment_metadata.c and write_mef_segment_metadata.c all the others are with "MEX completed successfully" output

jezdip1 commented 1 year ago

Plain Text.txt the crashlog

MaxvandenBoom commented 1 year ago

Hi, sorry to hear it doesn't work for you.

I just tried the existing binaries from v.1.2.2 on Linux (Debian GNU/Linux 11), and for me the sampledata loads just fine: matmef_sampledata_working We also use matmef on Mac and Windows in our lab to load iEEG data on a daily basis without any problems, so I'm curious to see what goes wrong on your system.

I did notice that I hadn't updated the Linux binaries for a bit, so I recompiled them (Debian GNU/Linux 11, gcc version 10.2.1, Matlab R2022b) and updated matmef here (now on release v1.2.3) to include the latest pre-compiled binaries.

Could you clone matmef repo again and try using the already included binaries? Also, could you show me what matlab command you are using to load the data?

Best, Max

jezdip1 commented 1 year ago

hmm, no change image tried the new clone, as well as release-1.2.3.zip download behaving the same. could it be related I am using x2go remote access to my lab PC?

jezdip1 commented 1 year ago

pffff, needs to be the absolute path for the sample_data. now it works, thanks

MaxvandenBoom commented 1 year ago

I see, I was not aware.

A relative path in Windows works just fine: matmef_relpath_windows

However doing the same thing on a Mac crashes it as well. I'm going to change the title and keep this issue open until I get around to fixing it.

Thanks for reporting!