Skip to content

Conversation

@cbrnr
Copy link
Contributor

@cbrnr cbrnr commented Apr 8, 2022

Fixes #10499.

@cbrnr
Copy link
Contributor Author

cbrnr commented Apr 8, 2022

@larsoner feel free to merge (or let me know if you want me to change anything).

@larsoner larsoner merged commit 493418e into mne-tools:main Apr 8, 2022
@larsoner
Copy link
Member

larsoner commented Apr 8, 2022

Thanks @cbrnr !

@cbrnr cbrnr deleted the metainfo-combine-channels branch April 9, 2022 07:37
larsoner added a commit to wmvanvliet/mne-python that referenced this pull request Apr 19, 2022
* upstream/main: (40 commits)
  FIX: Flake (mne-tools#10540)
  FIX: Correct link (mne-tools#10536)
  DOC: Update installers (mne-tools#10535)
  ENH: Add dark mode to website (mne-tools#10523)
  WIP: Copy BEM surfaces by default (don't symlink) (mne-tools#10531)
  Avoid lowpass=0 in brainvision data (mne-tools#10517)
  DOC: Update installers [skip azp] [skip actions] (mne-tools#10528)
  FIX: Fix for old build (mne-tools#10527)
  Fix line noise at wrong frequencies (mne-tools#10525)
  FIX : read fids in eeglab (mne-tools#10521)
  MAINT: Prefer PySide6 in testing (mne-tools#10513)
  ENH: Add overview_mode support (mne-tools#10501)
  MRG: Updates for qtpy in mne-qt-browser (mne-tools#10509)
  BUG: Fix bug with themes on macOS (mne-tools#10500)
  MAINT: Bump installer links (mne-tools#10511)
  Add metadata to combine_channels (mne-tools#10504)
  MAINT: Standardize tests (mne-tools#10502)
  CI: Test circle (mne-tools#10506)
  ENH: Use HiDPI splash screen on HiDPI screens (mne-tools#10503)
  WIP,MNT: Add support for QtPy (mne-tools#10430)
  ...
@cbrnr
Copy link
Contributor Author

cbrnr commented Apr 21, 2022

Could we backport this fix @larsoner?

agramfort added a commit that referenced this pull request Apr 21, 2022
* Add metadata to combine_channels

* Update mne/channels/channels.py

Co-authored-by: Alexandre Gramfort <[email protected]>

* Add changelog entry

* Add test

Co-authored-by: Alexandre Gramfort <[email protected]>
@agramfort
Copy link
Member

done

@cbrnr
Copy link
Contributor Author

cbrnr commented Apr 21, 2022

Thanks @agramfort!

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.

mne.channels.combine_channels(epochs) drops metadata

3 participants