Skip to content

Finalize json files for outputs#8

Merged
mattcieslak merged 7 commits into
mainfrom
single-metadata-file
Jun 16, 2025
Merged

Finalize json files for outputs#8
mattcieslak merged 7 commits into
mainfrom
single-metadata-file

Conversation

@mattcieslak

Copy link
Copy Markdown
Contributor

No description provided.

@mattcieslak
mattcieslak requested review from tien-tong and tsalo June 16, 2025 13:41
Comment thread freesurfer_post/data/surfacestats.json Outdated
mattcieslak and others added 3 commits June 16, 2025 10:00
Co-authored-by: Taylor Salo <salot@pennmedicine.upenn.edu>
Comment thread freesurfer_post/data/surfacestats.json Outdated
Comment thread freesurfer_post/data/surfacestats.json Outdated
"Description": "Number of surface vertices included in this parcel. This is a native surface, so the number of vertices in total will differ across scans."
},
"surfarea": {
"Description": "Sum of the areas of the triangles included in the parcellation",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should probably be consistent throughout, either parcel or ROI

Comment thread freesurfer_post/interfaces/interfaces.py Outdated
Comment thread freesurfer_post/data/surfacestats.json Outdated
Comment thread freesurfer_post/data/surfacestats.json Outdated
Comment thread freesurfer_post/data/surfacestats.json Outdated
Comment thread freesurfer_post/data/surfacestats.json Outdated
Comment thread freesurfer_post/data/surfacestats.json Outdated
Comment thread freesurfer_post/data/surfacestats.json Outdated
mattcieslak and others added 2 commits June 16, 2025 11:43
Co-authored-by: Tien Tong <35613222+tientong98@users.noreply.github.com>
@mattcieslak
mattcieslak merged commit a724154 into main Jun 16, 2025
1 check passed
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