Skip to content

Conversation

@larsoner
Copy link

Chooses the sign of the max ori to match whatever the source normal is defined as in the forward (and thus in original src).

@larsoner
Copy link
Author

This is what i get in the updated example, suggesting that the signs are now properly flipped:

screenshot from 2018-09-25 10-24-26

screenshot from 2018-09-25 10-24-28

For reference this is our current output:

http://mne-tools.github.io/dev/auto_examples/inverse/plot_lcmv_beamformer.html

@larsoner
Copy link
Author

@britta-wstnr you might be interested in this output

@wmvanvliet wmvanvliet force-pushed the cleanup_beamformer branch 2 times, most recently from 833e64b to efa3c40 Compare September 28, 2018 14:51
@wmvanvliet
Copy link
Owner

Tnx @larsoner !

@larsoner larsoner deleted the cb branch October 1, 2018 10:46
wmvanvliet pushed a commit that referenced this pull request Dec 2, 2021
* make contrib guide more friendly

* move icons to beginning

Co-authored-by: Richard Höchenberger <richard.hoechenberger@gmail.com>

* more consistent icon style

* Additions to contrib guide PR (#6)

* make line about fork clearer

* minimize white space and improve wording

* improve some wording

Co-authored-by: Richard Höchenberger <richard.hoechenberger@gmail.com>
Co-authored-by: Britta Westner <britta.wstnr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants