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

Add multi-channel conversion transforms for brats23 #8112

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

dani-capellan
Copy link

Hi team!

I added some new transforms to convert from BraTS2023 labels to multi-channel - both when performing region-based and label-based training. I thought it could be useful because right now the only one existing in monai.transforms (ConvertToMultiChannelBasedOnBratsClasses) is based on BraTS2018 labels, which have changed over the past editions. If you find this useful I could also try to implement them for BraTS2024, although here we have more heterogenity across the different tasks.

New classes:

  • ConvertToMultiChannelBasedOnBrats23Classes: returns labels -> regions -> multi-channel tensors. Simulates the previous mentioned class (ConvertToMultiChannelBasedOnBratsClasses)
  • ConvertToMultiChannelBasedOnBrats23ClassesNoReg: returns labels -> multi-channel tensors

I've quickly tested the changes by running it in some experiments I'm working on, but I haven't run any integration tests yet. Please let me know if this is helpful. I'm happy to assist further with the implementation or help finalize the pull request.

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

Copy link
Contributor

@KumoLiu KumoLiu left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!
Instead of introducing several new transforms here, I would suggest we can extend the existing one such as add an argument which can refer to what dataset you want to convert.
Also we may need to include the test for these update.
Does this makes sense to you? Thanks again!

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.

2 participants