mlActMan instead of mlActAuto in hmrR_MotionArtifactByChannel?

[Sabrina Brigadoi]

Hi,

there seems to be a problem with the hmrR_MotionArtifactByChannel function (and also the hmrR_MotionArtifact one): it does not consider mlActAuto but only mlActMan, which means that it will run also on channels what were pruned by the enPruneChannel function, screwing up the results. Was this done on purpose? 

I tried to change the input variable name to the function to have a quick fix, but it doesn't work, there is probably something else that has to be modified as well?

Thanks,

Sabrina

 

[Stephen Tucker]

As far as I can tell the excluded channels should be the OR of mlActAuto and mlActMan, and this is a mistake. I will work on a fix to the functions.

Stephen

--
You received this message because you are subscribed to the Google Groups "Homer-fNIRS and AtlasViewer Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email to homer-fnirs...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/homer-fnirs/CA%2BxoYy8j56V8Y%3DY6cOSfo0n9fc_sct%2BOCjDcs3LYuB-LEJ88HQ%40mail.gmail.com.

[Stephen Tucker]

Correcting myself-- it is logical-and, not logical-or. A fix should be available soon.

[Stephen Tucker]

Hi Sabrina,

The latest version of Homer has implemented a fix for this issue.

Please let me know if the functions now behave how you would expect.

Best,

Stephen

You received this message because you are subscribed to a topic in the Google Groups "Homer-fNIRS and AtlasViewer Community" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/homer-fnirs/05YvW0bezKk/unsubscribe.
To unsubscribe from this group and all its topics, send an email to homer-fnirs...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/homer-fnirs/bccf5aed-9b71-4c8b-9098-b419c346ea4an%40googlegroups.com.

[Adam Andersen]

Hi Stephen,

I'm getting this error message with the latest version of Homer, which I didn't before:

Brace indexing is not supported for variables of this type.

Error in hmrR_MotionArtifactByChannel (line 100)
    if isempty(tIncMan{iBlk})

Kind regards,

Adam

To view this discussion on the web visit https://groups.google.com/d/msgid/homer-fnirs/CAPokv69rrYeFxv9cW0cacYEcFhgDHyO%3DM5Q9_31p_OwCdjB8oA%40mail.gmail.com.

[Stephen Tucker]

Hi Adam,

Try deleting the generated .mat files and creating a new processing stream. Some used functions have changed.

Best,

Stephen

To view this discussion on the web visit https://groups.google.com/d/msgid/homer-fnirs/CAPu%3D%3DLwQmOKbjcrxz7gSxdec%3DdFzDtMB%2BeGa-EytuseufbzgXA%40mail.gmail.com.