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

BAMF FDG-Avid Breast Tumor #79

Closed
wants to merge 13 commits into from

Conversation

jithenece
Copy link

No description provided.

@vanossj
Copy link
Contributor

vanossj commented May 2, 2024

@LennyN95 What is the right way to have multi modality inputs into the NNUnetRunner? We made a custom runner, but it doesn't feel like that should be required.

@LennyN95
Copy link
Member

LennyN95 commented May 2, 2024

@vanossj if it's an nnunet that takes e.g. multiple modalities interchangeably but only one input at a time, then you could use the | operator like in nifti:mod=mr|ct.
However, if you have multiple inputs (as your implementation suggests), the current version of the NNUnet runner indeed isn't capable of handling this. We are planning to update this (however, on a lower prior). So, in this case, a custom module (you can, of course, start from our implementation and simply make the desired modifications) would be the fastest.

@vanossj
Copy link
Contributor

vanossj commented May 6, 2024

@LennyN95 Could you add a roi for FDG-avid tumor? (NCIt C168968). if you have an alternative sct code that would work too

@LennyN95
Copy link
Member

LennyN95 commented May 7, 2024

@jithenece @vanossj As per your suggestion, I added FDG_AVID_TUMOR to SegDB.

@jithenece
Copy link
Author

@LennyN95 could you add BREAST to segdb as it is required for generating roi=BREAST+FDG_AVID_TUMOR.

@LennyN95
Copy link
Member

@jithenece done, added BREAST as new segmentation to SegDB.

@jithenece
Copy link
Author

I tried to test this but seems to fail. Does this require updating base image mhubai/base:latest.
sha256:d72b350c7f98e5f2460297d6059a70f967091c9ed11728ab3bbc7f9b32b65d2e

@jithenece done, added BREAST as new segmentation to SegDB.

@jithenece
Copy link
Author

closing this as this will be done in another pull request

@jithenece jithenece closed this Jul 2, 2024
@jithenece jithenece deleted the bamf_nnunet_pet_ct_breast branch July 2, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants