Skip to content

Electrophysiology chunker MEG improvements#1418

Open
MaximeBICMTL wants to merge 2 commits into
aces:mainfrom
MaximeBICMTL:remove-meg-noise
Open

Electrophysiology chunker MEG improvements#1418
MaximeBICMTL wants to merge 2 commits into
aces:mainfrom
MaximeBICMTL:remove-meg-noise

Conversation

@MaximeBICMTL
Copy link
Copy Markdown
Contributor

@MaximeBICMTL MaximeBICMTL commented Apr 21, 2026 •
edited
Loading

Description

This PR contains the following changes:

  • Remove the noise from MNE MEG raw object before writing values. This is required for MEG because the noise is actually much more ample than the signal, so visualizing without noise reduction is meaningless.
  • Pass MNE's raw object directly to the chunking.write_chunk_directory function, for MEG this allows to avoid reading the acquisition file twice (it does not change anything for EDF and EEGLAB).
  • Remove preload=False from MNE's read_raw functions since False is the default.

@github-actions github-actions Bot added Language: Python Issue or PR related to the Python codebase Package: Electrophysiology chunker PR or issue related to the electrophysiology chunker labels Apr 21, 2026
@MaximeBICMTL MaximeBICMTL added the Complexity: Simple Issue or PR that should be simple to implement, review, or test label Apr 21, 2026
@cmadjar cmadjar requested a review from jeffersoncasimir May 12, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jeffersoncasimir jeffersoncasimir Awaiting requested review from jeffersoncasimir

Assignees

No one assigned

Labels

Complexity: Simple Issue or PR that should be simple to implement, review, or test Language: Python Issue or PR related to the Python codebase Package: Electrophysiology chunker PR or issue related to the electrophysiology chunker

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MaximeBICMTL