Nothing wrong with getSegmentationType and hasnt caused any issues (i think)

[jjjermiah]

just wondering whatre the odds that the csv file might have
both 'RTSTRUCT' AND 'SEG' in modality column, and maybe we could check for that and throw an error?

Projects:

https://github.com/bhklab/readii/blob/eaeb904eb42c8abb1bef02e9d2921a8d8f503648/src/readii/metadata.py#L210-L256

[strixy16]

It technically is a possibility. I implemented this function so that we don't have to pass the modality in as an argument and wondered about this instance too at the time.

We could reimplement the option to set which modality to use and if it's not supplied we extract it automatically. Then if this function fails, we tell the user to set the modality when running the command.

In the orcestra pipeline it shouldn't be an issue because it's setup to specifically download RTSTRUCTS.