search

sccn / EEG-BIDS

21 stars 17 forks source link

`bidsstats` not defined #221

Closed neuromechanist closed 2 days ago

neuromechanist commented 3 days ago

I am trying to export the processed HBN Release 3 using the new (and the old) EEG-BIDS and I get the following error: bids_export error: undefined argument 'bidsstats'

This error occurs in bids_export line 327.

arnodelorme commented 3 days ago

Let me look into that.

On Jul 4, 2024, at 16:43, Seyed (Yahya) Shirazi @.***> wrote:

I am trying to export the processed HBN Release 3 using the new (and the old) EEG-BIDS and I get the following error: bids_export error: undefined argument 'bidsstats' This error occurs in bids_export line 327. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>

arnodelorme commented 2 days ago

I think you might not be using the latest version. Line 327 does not have 'bidsstats'. In fact there is no mention of 'bidsstats' in the file. Let me know.

https://github.com/sccn/EEG-BIDS/blob/64ed249d2a315849b9ec0e5c84df0aa0f397bd64/bids_export.m#L327

neuromechanist commented 2 days ago

I used the EEG-BIDS from the EEGLAB plug-in browser. Line 327 reads:

if isstr(opt), error(opt); end

I need to keep the pipeline limited to stable releases to ensure reproducibility.

Yes, there is no bidsstats option, which makes me wonder if this argument is coming from the upstream functions. I will take a look.

arnodelorme commented 2 days ago

The best version of this plugin is the dev version. When your pipeline is stable, we can issue a new release of the plugin which you can reference in the paper.