[snirf] Improve support to SNIRF

[Edouard2laire]

This PR adds support for the new measurement list (added in v1.1) and overall provides better support to the SNIRF specifications

The snirf specification can be found here: https://github.com/fNIRS/snirf/blob/master/snirf_specification.md

Todo:

Reading SNIRF :

• read snirf data with dOD / dHb data
• Read coordinate system ( 0 % done)
• Suport 2D array in AUX
• Read dataUnit (https://github.com/fNIRS/snirf/blob/master/snirf_specification.md#nirsidatajmeasurementlistsdataunit)
• Support time vector as Start + sample spacing (https://github.com/fNIRS/snirf/blob/master/snirf_specification.md#nirsidatajtime)
• Round wavelength
• Add error for unsupported data format
• Apply dataOffset (https://github.com/fNIRS/snirf/blob/master/snirf_specification.md#nirsidatajdataoffset)
• Add support for MeasurmentLists
• Read MeasurementDate / MeasurementTime
• Read ManufacturerName / Model

Writting snirf :

• write snirf data with dOD / dHb data ( 0 % done)

Once all the todo are done; we should be fairly complient with the snirf standard.

[Edouard2laire]

@rcassani what would be the best way to handle the optional flag "coordinateSystem" ?
https://github.com/fNIRS/snirf/blob/master/snirf_specification.md#nirsiprobecoordinatesystem

Notably, if it is "MNI152NLin2009bAsym" then we need to convert the coordinates from MNI to SCS ? I guess we should already have some function comming from the BIDS import to dead with that.

[rcassani]

@Edouard2laire, this is where the magic from MNI space to Subject space happens:

brainstorm3/toolbox/io/import_channel.m

Lines 376 to 377 in 85f367a

	%% ===== MNI TRANSFORMATION =====
	if ismember(FileFormat, {'ASCII_XYZ_MNI', 'ASCII_NXYZ_MNI', 'ASCII_XYZN_MNI', 'INTRANAT_MNI', 'BIDS-MNI-MM', 'BIDS-MNI-CM', 'BIDS-MNI-M'})

[Edouard2laire]

@Edouard2laire, this is where the magic from MNI space to Subject space happens:

brainstorm3/toolbox/io/import_channel.m

Lines 376 to 377 in 85f367a

	%% ===== MNI TRANSFORMATION =====
	if ismember(FileFormat, {'ASCII_XYZ_MNI', 'ASCII_NXYZ_MNI', 'ASCII_XYZN_MNI', 'INTRANAT_MNI', 'BIDS-MNI-MM', 'BIDS-MNI-CM', 'BIDS-MNI-M'})

yes; but this function is not called when doing import_data_snirf; right ? since the channel is defined during the import of the data ?

[Edouard2laire]

ok. i think the code is much cleaner now.

@rcassani, do you think you could help me fill the gap here:

% Read coordinate system
if isfield(jnirs.nirs.probe, 'coordinateSystem') && ~isempty(jnirs.nirs.probe.coordinateSystem)
    warning('Todo: apply coordinate system');
end

https://github.com/brainstorm-tools/brainstorm3/pull/756/files#diff-92d5b968a90e2607f65b091acae076d6c977f09b46025c07b190cdd00e26ad28R97-R99