Skip to content

added definition of beamformer to the glossary#6864

Merged
larsoner merged 2 commits intomne-tools:masterfrom
NataKozh:mne_glossary_enh_6321
Oct 1, 2019
Merged

added definition of beamformer to the glossary#6864
larsoner merged 2 commits intomne-tools:masterfrom
NataKozh:mne_glossary_enh_6321

Conversation

@NataKozh
Copy link
Copy Markdown
Contributor

@NataKozh NataKozh commented Oct 1, 2019

Added definition of beamformer to the glossary

@drammock

doc/glossary.rst Outdated
for a tutorial on how to manipulate such objects.

Beamformer
Beamformer is a popular source estimation approach that uses a set of spatial filters (beamformer weights) to compute time courses of sources which coordinates are predefined. Beamformer weight for a particular source is selected so it minimizes average reconstructed source power at a given location, subject to a unit gain constraint. See :class:`mne.beamformer.Beamformer’.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@britta-wstnr okay for you?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not all beamformers have the unit gain constraint. I would just remove the sentence Beamformer weight for a ... constraint.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 1, 2019

Codecov Report

Merging #6864 into master will decrease coverage by 1.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6864      +/-   ##
==========================================
- Coverage   89.63%   88.39%   -1.25%     
==========================================
  Files         425      425              
  Lines       76004    76003       -1     
  Branches    12398    12398              
==========================================
- Hits        68125    67180     -945     
- Misses       5102     6044     +942     
- Partials     2777     2779       +2

@larsoner
Copy link
Copy Markdown
Member

larsoner commented Oct 1, 2019

Thanks @NataKozh

@larsoner larsoner merged commit ba5c83e into mne-tools:master Oct 1, 2019
@NataKozh NataKozh deleted the mne_glossary_enh_6321 branch October 1, 2019 17:37
@drammock drammock mentioned this pull request Oct 1, 2019
11 tasks
alexrockhill pushed a commit to alexrockhill/mne-python that referenced this pull request Oct 3, 2019
* added definition of beamformer to the glossary

* Glossary changes: definitions of beamformer and digitization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants