mne-tools/mne-python — pull request #643 by rgoj (2013-06-13 16:13:52+00:00)
Title: FIX Move lcmv normal orientation picking

Normal orientation picking is now done on `W` instead of on `G`. This ensures consistency with the minimum_norm implementation of picking the normal orientation. Addresses discussion in issue #635.

For future reference - I think it might be worthwhile (although I'm less and less sure it would be) to consider adding the option of choosing source orientation on the gain matrix and therefore ensuring this is a scalar beamformer implementation.

Also note that now (in the test modified in this commit) the results from the normal orientation never differ from results of the free orientation solution by more than 0.4 (compared to 0.8 before moving the orientation picking). I also checked that when the picking is moved after noise normalization (i.e. around [line 176](https://github.com/rgoj/mne-python/blob/c09f97018a06a5e659e52009cdc57e1aafe17d99/mne/beamformer/_lcmv.py#L176)), then the normal solution is always smaller than the free orientation one. Interesting how big a difference numerical issues caused here.