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

[MAINT] update listing and add validation #485

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

Remi-Gau
Copy link
Contributor

@Remi-Gau Remi-Gau commented Mar 3, 2025

should fix: #483 (comment)

@@ -82,3 +82,4 @@ mrs_fmrs Functional MRS data involving a pain stimulus task from 15 subjects [li
2d_mb_pcasl Siemens 2D MultiBand Multi-delay PCASL (m0 and noRF included within timeseries) [link](https://osf.io/xrkc4/) [@aptinis](https://github.com/aptinis) anat, fmap, perf T1w, asl, aslcontext, epi
xeeg_hed_score EEG and iEEG data with annotations of artifacts, seizures and modulators using HED-SCORE [@dorahermes](https://github.com/dorahermes) anat, eeg, ieeg T1w, channels, coordsystem, eeg, electrodes, events, ieeg
dwi_deriv exemplifies the storage of diffusion MRI derivates that may be generated on the Siemens XA platform. dwi dwi
pheno004 dataset containing a mismatch of subjects with MRI data and tabular phenotypic data to be considered a whole dataset of participants for the sake of the participants.tsv file [@ericearl](https://github.com/ericearl) anat T1w, ace, demographics
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@effigies
seems pybids is indexing the content of phenotype folder and parsing the files trying to get some suffixes: hence why we have ace and demographics in the suffix column here.

is that something we want in terms of behavior for pybids?

In am not convinced it is but maybe I am missing something.

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.

1 participant