[IO] Load kernel data

[Edouard2laire]

Hello,

Those are minor fix to load nirs KERNEL data. Also remove the automatic computation of HbT as its not really usefull and can be confusing for users to see HbT even if its not present in their data.

This PR also adds support for time-domain nirs (without importing additional moments for now as i dont know how to process them yet).

Also fix unit for raw data (mV)

[rcassani]

Also remove the automatic computation of HbT as its not really usefull and can be confusing for users to see HbT even if its not present in their data.

Do you thing adding a warning or other kind visual aid would help confused users?

Completely removing HbT from the NIRS montage seems to me a bit too drastic. Having HbT in the NIRS montage has been in Brainstorm for more than 10 years. As montage, it is only visual, no files are created, nor channels added in the channel file. Unless the montage is explicitly applied on the data by the user, in that case hopefully they would know what they are doing.

[Edouard2laire]

The issue is also that it doesnt work:

Then, if you select a channel and press =, there is no HbT:

Becauuse, we already compute HbT, inside nirstorm, i think this was never used. If we want the feature it should be an option like for GFP but i dont have time to add it for now (and i dont think anyone needs it -- we can check if anyone ask)

[rcassani]

The issue is also that it doesnt work

Oh, that was unexpected. Let me give it a try to fix otherwise we can remove it as suggested.
Please send me NIRS file to reproduce the images shown.

Becauuse, we already compute HbT, inside nirstorm, i think this was never used.

That's true, so that montage does not have any effect in recordings that have already [HbT]

[Edouard2laire]

i'll send you a file tomorrow. but i am not sure its worth fixing :) i would prefer having #897 or #894 merged before