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

Missing items from TotalSegmentator v2 mapping #42

Closed
lassoan opened this issue Oct 27, 2023 · 4 comments · Fixed by #43
Closed

Missing items from TotalSegmentator v2 mapping #42

lassoan opened this issue Oct 27, 2023 · 4 comments · Fixed by #43

Comments

@lassoan
Copy link
Owner

lassoan commented Oct 27, 2023

These labels are missing from the mapping csv file:

  • vertebrae_S1
  • thyroid_gland
@lassoan
Copy link
Owner Author

lassoan commented Oct 27, 2023

@fedorov The TotalSegmentator v2 mapping is almost complete, just two labels are missed (see above). Could you add their mapping to the .csv file? Thank you! If it is done then it could make sense to submit the .csv to the TotalSegmentator repository, too.

@fedorov
Copy link
Contributor

fedorov commented Oct 27, 2023

If it is done then it could make sense to submit the .csv to the TotalSegmentator repository, too.

I agree. I will do it once it is updated here first.

@fedorov
Copy link
Contributor

fedorov commented Oct 27, 2023

@lassoan can you please let me know what are the label numbers for those 2 missing labels, and where they are located in TotalSegmentator repo? I tried and failed in looking for those.

@lassoan
Copy link
Owner Author

lassoan commented Oct 27, 2023

I don't think there are fixed label values (each task uses different label values). You can find find S1 here: https://github.com/wasserth/TotalSegmentator/blob/cad2230bc2756c555b13afa5282019d1c87b7a5a/totalsegmentator/map_to_binary.py#L138 the other one is in the same file.

We can probably remove the label numbers from the .csv but you can ask Jakob to confirm.

fedorov added a commit to fedorov/SlicerTotalSegmentator that referenced this issue Oct 30, 2023
lassoan pushed a commit that referenced this issue Oct 31, 2023
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 a pull request may close this issue.

2 participants