Stephen Tucker

This is great. Our documentation that is similar is in need of an update.

@dboas I will leave this to you to close.

Thank you for this excellent contribution. I have the following comments on the code: It would be better to refactor the `isExpectedClass` function and any other repeated code into a...

@vijayiyer05 Ah, okay. I am only concerned with the dependence of this code. Which is > 2017a?

I strongly think we should shelve any changes that require an update beyond 2017b for now. Anything else would be great to get merged.

Has this been implemented since @mayucel ?

Need to add support for units in Homer3. Upcoming version of SNIRF will also incorporate units for aux and data channels. As you point out, we don't even support `snirf.metaDataTags.tags.TimeUnit`

The assumption that sampling rates are constant throughout the signal is pretty baked in to Homer processing. The tool you describe would be useful. You could add it as a...

There may be one built in under Tools drop down towards the bottom? Can't remember right now.

I am really happy to see contributions to the project but this is a rather large change to the GLM function, which is already bloated in functionality. Is it possible...