qurit / qurit/rt-utils

add ability to retrieve ROI names in a case insensitive way

Open
#44 2 comments 0 reactions 0 assignees View on GitHub

@raffaelespinoni is already working on this.

Since Feb 22, 2022.

  • #47 by @raffaelespinoni — open
Dominant language
Python
Stars
256
Forks
75
PR merge metrics
No merged PRs in 30d

Description

HI,
i'm experiencing problem with low quality ROI labels (written like this: Heart, heart, HEART),
maybe an option to extract case insensitive ROIs could be useful to other also

I have a quick_fix in rtstruct.py:98
if structure_roi.ROIName.casefold() == name.casefold():

maybe a more general option while using ".get_roi__mask_by_name" could be better

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading rtstruct.py around line 98 and the get_roi__mask_by_name entry point to understand the existing ROI-name comparison. Review linked pull request #47 for the work already underway. Done should provide an agreed way to retrieve ROI names without case-sensitive matching.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.