Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added definition of beamformer to the glossary #6864

Merged
merged 2 commits into from
Oct 1, 2019

Conversation

NataKozh
Copy link
Contributor

@NataKozh NataKozh commented Oct 1, 2019

Added definition of beamformer to the glossary

@drammock

doc/glossary.rst Outdated
@@ -24,6 +24,9 @@ general neuroimaging concepts. If you think a term is missing, please consider
object class and :ref:`tut-annotations`
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
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
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

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
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