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

validator for reproin #26

Open
satra opened this issue Jan 19, 2019 · 2 comments
Open

validator for reproin #26

satra opened this issue Jan 19, 2019 · 2 comments
Labels
enhancement New feature or request good-for-hackathon Good issue/enhancement to tackle at a hackathon

Comments

@satra
Copy link

satra commented Jan 19, 2019

at present it's hard for people to debug via heudiconv whether the reproin naming scheme they adopted on the scanner is valid. would it be possible to add a validator to heudiconv that can simply go through the dicom grouping and check if the names are reproin compliant. or have heudiconv give better error messages when reproin heuristic is used.

@yarikoptic
Copy link
Member

Yeah, I wanted something like that too. Eventually we could compliment it with ocr of the photo from console display ;-)

@yarikoptic yarikoptic added enhancement New feature or request good-for-hackathon Good issue/enhancement to tackle at a hackathon labels May 3, 2019
@yarikoptic
Copy link
Member

note: not a validator but rather "constructor" was created - https://github.com/NPACore/reproin-namer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good-for-hackathon Good issue/enhancement to tackle at a hackathon
Projects
None yet
Development

No branches or pull requests

2 participants