feat: add interactive CSV column mapper to DICOM standard#439
feat: add interactive CSV column mapper to DICOM standard#439JoshuaSiraj wants to merge 6 commits intoJoshuaSiraj/nifti-indexfrom
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
I like dicommap. I feel like map is too general and could be used for something in the future. |
| ) | ||
| @click.option( | ||
| "-y", | ||
| "--yes", |
There was a problem hiding this comment.
This is a really vague flag, maybe change to interactive?
@skim2257 @strixy16
Should I change the name of the files from dicommap.py to map.py?