Skip to content

Conversation

erikglee
Copy link
Contributor

(1) Made adjustments to _get_age_from_tsv and age_to_months so that _get_age_from_tsv tries to pass a float (instead of an int) to age_to_months. The main goal behind this is to be able to give greater precision to age measures that are reported in years.
(2) Made adjustments to parse_bids_for_age_months and _get_age_from_tsv to have unique handling of age if a sessions.tsv is the source of age information. The previous functionality of _get_age_from_tsv when scans.tsv files were used was to try to grab age information from the first acquisition mentioned in the file. This current update instead has _get_age_from_tsv try to grab age from the first "anat" file in the scans.tsv. This change attempts to provide more precise age measures in the situation where many data elements (MRI, EEG, accelerometry, etc.) are collected at different times but still utilize the same scans.tsv file.

@scott-huberty
Copy link
Contributor

scott-huberty commented Sep 12, 2024

Thanks @erikglee !🙏

xref #389 .. Does this close (supercede) that PR?

@mgxd
Copy link
Collaborator

mgxd commented Sep 20, 2024

@erikglee thanks for the patch - I went ahead and made a few changes on top of this in #395, so closing this in favor of that.

@mgxd mgxd closed this Sep 20, 2024
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.

3 participants