search

sccn / lsl_archived

Multi-modal time-synched data transmission over local network
242 stars 133 forks source link

LabRecorder default_config needs OnlineSync example #186

Open cboulay opened 7 years ago

cboulay commented 7 years ago

The LabRecorder app has code to parse the config file. Unfortunately, this fails if the config file is missing an OnlineSync section. Also, adding OnlineSync=[] does not allow it to finish.

@dmedine , can you please provide a config file example for OnlineSync and/or fix the parsing code so it doesn't error-out if OnlineSync is not provided or empty?

dmedine commented 7 years ago

@Chadwick, thanks for cleaning up the repository issues page so much today! It definitely needed some attention.

I will try to put some attention to the online sync stuff over the next few days/weeks. I ran one fairly robust test of this back in December and did not get the results I expected which is why I didn't put any time into documentation (don't know what to document yet!).

Anyway, this whole thing is a hanging chad (no pun intended!). In the meantime I will try to deal with this parsing bug.

On 30.03.2017 02:35, Chadwick Boulay wrote:

The LabRecorder app has code to parse the config file. Unfortunately, this fails if the config file is missing an OnlineSync section. Also, adding |OnlineSync=[]| does not allow it to finish.

@dmedine https://github.com/dmedine , can you please provide a config file example for OnlineSync and/or fix the parsing code so it doesn't error-out if OnlineSync is not provided or empty?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sccn/labstreaminglayer/issues/186, or mute the thread https://github.com/notifications/unsubscribe-auth/ADch7tVTtnDq7zx_7BeYR7CnWsXCxnmBks5rqvjqgaJpZM4MtuZB.

dmedine commented 7 years ago

Okay. I fixed the bug in the source code. I also eliminated MATLABImporter from the repository because it is simply too much work to maintain load_xdf.m in two places. Go to https://github.com/sccn/xdf for the latest version of that.

It is also too much work for me (and soon will be impossible for me) to upload releases to the SCCN ftp anymore. The Swarz Center will need to get someone to take care of this or else the releases will have to move. I still have access to my old UCSD stuff, but they will cut me off any day now and it is extremely labor intensive for me to upload releases every time some small fix is made.

On 30.03.2017 02:35, Chadwick Boulay wrote:

The LabRecorder app has code to parse the config file. Unfortunately, this fails if the config file is missing an OnlineSync section. Also, adding |OnlineSync=[]| does not allow it to finish.

@dmedine https://github.com/dmedine , can you please provide a config file example for OnlineSync and/or fix the parsing code so it doesn't error-out if OnlineSync is not provided or empty?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sccn/labstreaminglayer/issues/186, or mute the thread https://github.com/notifications/unsubscribe-auth/ADch7tVTtnDq7zx_7BeYR7CnWsXCxnmBks5rqvjqgaJpZM4MtuZB.