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

bugfix: subfield+tissue for new atlases, and tsv files #208

Merged
merged 2 commits into from
Aug 14, 2022
Merged

Conversation

akhanf
Copy link
Member

@akhanf akhanf commented Aug 13, 2022

Two related fixes:

  1. New atlases were still using the bigbrain dseg.tsv for generating the
    subfield volumes.tsv files. These have now been added.

  2. The way the postproc tissue labels (dg, srlm, cyst) were being added
    to the subfields was also hardcoded for the bigbrain atlas. Have now
    added a section in the config file to customize what labels these map
    to, so that e.g. the dentate gyrus is merged into the existing label
    number, and that the SRLM/Cyst labels do not conflict with the labels,
    and are maximal (so they override the others). Previously the magdeburg
    subiculum label (8) was same as cyst, so these became merged.

Two related fixes:

1. New atlases were still using the bigbrain dseg.tsv for generating the
subfield volumes.tsv files. These have now been added.

2. The way the postproc tissue labels (dg, srlm, cyst) were being added
to the subfields was also hardcoded for the bigbrain atlas. Have now
added a section in the config file to customize what labels these map
to, so that e.g. the dentate gyrus is merged into the existing label
number, and that the SRLM/Cyst labels do not conflict with the labels,
and are maximal (so they override the others). Previously the magdeburg
subiculum label (8) was same as cyst, so these became merged.
@akhanf akhanf added the bug Something isn't working label Aug 13, 2022
@akhanf
Copy link
Member Author

akhanf commented Aug 14, 2022

tested and working, so merging this in

@akhanf akhanf merged commit 3cef610 into master Aug 14, 2022
@akhanf akhanf deleted the atlas_tsv branch August 14, 2022 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant