-
Notifications
You must be signed in to change notification settings - Fork 1.4k
ENH: Support TD data #11064
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
Draft
larsoner
wants to merge
44
commits into
mne-tools:main
Choose a base branch
from
larsoner:feature/TD-nirs_snirf
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
ENH: Support TD data #11064
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
3f875d3
adding the new snirf data types to the constants
Zahra-M-Aghajan cc92adc
more places to add the data types and constants
Zahra-M-Aghajan 0d2fe87
adding constants for the allowed dataTypeLabels for the processed dat…
Zahra-M-Aghajan 21ecb90
updating the snirf reader to accept other data types: gated histogram…
Zahra-M-Aghajan 4491064
setting the formatter line length
Zahra-M-Aghajan dbdcebc
the reordering of the channels does not make much sense, need to inquire
Zahra-M-Aghajan 80de954
Merge remote-tracking branch 'upstream/main' into feature/TD-nirs_snirf
Zahra-M-Aghajan 32c5496
Merge remote-tracking branch 'upstream/main' into feature/TD-nirs_snirf
larsoner a35822a
MAINT: Simpler with .item
larsoner c2c217c
WIP: Constants
larsoner ac52bc0
FIX: Flake
larsoner da275f1
FIX: Consistent
larsoner 3986b96
FIX: Missed
larsoner ce8b3af
FIX: Better code
larsoner 1244add
FIX: Scale
larsoner 3944254
FIX: Missing
larsoner 967dddd
FIX: Flake
larsoner e97367d
FIX: Syntax
larsoner 4b963cd
Merge remote-tracking branch 'upstream/main' into feature/TD-nirs_snirf
larsoner 15e2878
Merge remote-tracking branch 'upstream/main' into feature/TD-nirs_snirf
larsoner d26ff8a
FIX: shape
larsoner f84fe27
FIX: Slash
larsoner 50f3064
FIX: Bad merge
larsoner 495bdf7
Merge branch 'main' into feature/TD-nirs_snirf
larsoner a076e23
Merge remote-tracking branch 'upstream/main' into feature/TD-nirs_snirf
larsoner 698d2a6
FIX: Add tests
larsoner d3b75ab
FIX: Reg
larsoner b1cf310
Merge branch 'main' into feature/TD-nirs_snirf
larsoner 0c4f886
MAINT: Revert
larsoner 34c36f5
Merge remote-tracking branch 'upstream/main' into feature/TD-nirs_snirf
larsoner b5e9967
Empty commit for credit
larsoner 34dffbf
DOC: Change
larsoner d62dbb1
MAINT: Mailmap
larsoner 880356e
FIX: SI
larsoner 4f0e8e6
FIX: Oops
larsoner a5c97b8
FIX: Oops
larsoner fba1ffa
FIX: idx
larsoner 9f4234d
WIP
larsoner b3b3f2e
FIX: Test
larsoner 9723621
Merge remote-tracking branch 'upstream/main' into feature/TD-nirs_snirf
larsoner 2b03f75
Merge remote-tracking branch 'upstream/main' into feature/TD-nirs_snirf
larsoner e9403d8
WIP: More constants
larsoner 6ee5068
FIX: Caps
larsoner b2b3716
FIX: More
larsoner File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -357,6 +357,7 @@ Yousra Bekhti <[email protected]> Yoursa BEKHTI <[email protected] | |
Yousra Bekhti <[email protected]> Yoursa BEKHTI <[email protected]> | ||
Yousra Bekhti <[email protected]> Yousra BEKHTI <[email protected]> | ||
Yousra Bekhti <[email protected]> yousrabk <[email protected]> | ||
Zahra M. Aghajan <[email protected]> Zahra M. Aghajan <[email protected]> | ||
Zhi Zhang <[email protected]> ZHANG Zhi <[email protected]> | ||
Zhi Zhang <[email protected]> ZHANG Zhi <[email protected]> | ||
Ziyi ZENG <[email protected]> ZIYI ZENG <[email protected]> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Added basic support for TD fNIRS data, by :newcontrib:`Zahra Aghajan`, :newcontrib:`Julien Dubois`, :newcontrib:`John Griffiths`, `Robert Luke`_, and `Eric Larson`_. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -32,6 +32,10 @@ | |
fnirs_fd_ac_amplitude="k", | ||
fnirs_fd_phase="k", | ||
fnirs_od="k", | ||
fnirs_td_gated_amplitude="k", | ||
fnirs_td_moments_intensity="k", | ||
fnirs_td_moments_mean="k", | ||
fnirs_td_moments_variance="k", | ||
csd="k", | ||
whitened="k", | ||
gsr="#666633", | ||
|
@@ -60,6 +64,10 @@ | |
fnirs_fd_ac_amplitude="V", | ||
fnirs_fd_phase="rad", | ||
fnirs_od="V", | ||
fnirs_td_gated_amplitude="AU", # counts | ||
fnirs_td_moments_intensity="AU", # counts | ||
fnirs_td_moments_mean="S", | ||
fnirs_td_moments_variance="S²", | ||
csd="V/m²", | ||
whitened="Z", | ||
gsr="S", | ||
|
@@ -88,6 +96,10 @@ | |
fnirs_fd_ac_amplitude="V", | ||
fnirs_fd_phase="rad", | ||
fnirs_od="V", | ||
fnirs_td_gated_amplitude="AU", | ||
fnirs_td_moments_intensity="AU", | ||
fnirs_td_moments_mean="S", | ||
fnirs_td_moments_variance="S²", | ||
csd="mV/m²", | ||
whitened="Z", | ||
gsr="S", | ||
|
@@ -117,6 +129,10 @@ | |
fnirs_fd_ac_amplitude=1.0, | ||
fnirs_fd_phase=1.0, | ||
fnirs_od=1.0, | ||
fnirs_td_gated_amplitude=1.0, | ||
fnirs_td_moments_intensity=1.0, | ||
fnirs_td_moments_mean=1.0, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. the values are in the range of nanoseconds, so this should be 1e-9 for mean and 1e-18 for variance maybe? |
||
fnirs_td_moments_variance=1.0, | ||
csd=1e3, | ||
whitened=1.0, | ||
gsr=1.0, | ||
|
@@ -151,6 +167,10 @@ | |
fnirs_fd_ac_amplitude=2e-2, | ||
fnirs_fd_phase=2e-1, | ||
fnirs_od=2e-2, | ||
fnirs_td_gated_amplitude=1.0, | ||
fnirs_td_moments_intensity=1.0, | ||
fnirs_td_moments_mean=1.0, | ||
fnirs_td_moments_variance=1.0, | ||
csd=200e-4, | ||
dipole=1e-7, | ||
gof=1e2, | ||
|
@@ -206,6 +226,10 @@ | |
fnirs_fd_phase="fNIRS (FD phase)", | ||
fnirs_od="fNIRS (OD)", | ||
hbr="Deoxyhemoglobin", | ||
fnirs_td_gated_amplitude="fNIRS (TD amplitude)", | ||
fnirs_td_moments_intensity="fNIRS (TD moment intensity)", | ||
fnirs_td_moments_mean="fNIRS (TD moment mean)", | ||
fnirs_td_moments_variance="fNIRS (TD moment variance)", | ||
gof="Goodness of fit", | ||
csd="Current source density", | ||
stim="Stimulus", | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JohnGriffiths @julien-dubois-k @Zahra-M-Aghajan let me know if you want a different URL to link from your name in the changelog
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
grifflab.com is perfect, thanks