Skip to content

[MRG] TUH: standardize channel name and define their positions - #593

Merged
bruAristimunha merged 13 commits into
braindecode:masterfrom
PierreGtch:tuh-define-channels
Apr 23, 2024
Merged

[MRG] TUH: standardize channel name and define their positions#593
bruAristimunha merged 13 commits into
braindecode:masterfrom
PierreGtch:tuh-define-channels

Conversation

@PierreGtch

Copy link
Copy Markdown
Collaborator

This gives the option to rename the channels of TUH to match the 1005 system and set the montage to get their positions

@PierreGtch
PierreGtch marked this pull request as ready for review April 13, 2024 19:17
@PierreGtch PierreGtch changed the title [WIP] TUH: standardize channel name and define their positions TUH: standardize channel name and define their positions Apr 13, 2024
@codecov

codecov Bot commented Apr 14, 2024

Copy link
Copy Markdown

Codecov Report

Merging #593 (035650f) into master (e15f28c) will decrease coverage by 0.26%.
The diff coverage is 33.33%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #593      +/-   ##
==========================================
- Coverage   87.14%   86.88%   -0.26%     
==========================================
  Files          65       65              
  Lines        5733     5758      +25     
==========================================
+ Hits         4996     5003       +7     
- Misses        737      755      +18     

@PierreGtch PierreGtch changed the title TUH: standardize channel name and define their positions [MRG] TUH: standardize channel name and define their positions Apr 15, 2024
@bruAristimunha

Copy link
Copy Markdown
Collaborator

Hey @agramfort,
I was wondering if we can merge this PR?

@bruAristimunha

bruAristimunha commented Apr 22, 2024

Copy link
Copy Markdown
Collaborator

The error is not related @PierreGtch, fixing in another PR

@bruAristimunha

Copy link
Copy Markdown
Collaborator

Solved @PierreGtch!

@bruAristimunha bruAristimunha mentioned this pull request Apr 22, 2024
@bruAristimunha
bruAristimunha merged commit a7e1103 into braindecode:master Apr 23, 2024
@bruAristimunha

Copy link
Copy Markdown
Collaborator

thank you @PierreGtch!

@bruAristimunha

Copy link
Copy Markdown
Collaborator

Hey @agramfort, I think you are busy, and as I had already done the review, I chose to apply the merge to fix the braindecode, which is linked and explained in the other PR.

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