search

cutright / IMRT-QA-Data-Miner

Scans a directory for IMRT QA results
MIT License
13 stars 5 forks source link

Merge ArcCheck and MapCheck parsers? #13

Closed mchamberland closed 4 years ago

mchamberland commented 4 years ago

As far as I can tell, MapCheck and ArcCheck reports are identical, except for the name at the top of the report (MapCheck or ArcCheck).

Should we merge the two parsers into one, then?

I have a few cases that even though we acquired two ArcCheck measurements, the report of the combined results show "MapCHECK" at the top of the report... We'd need to re-work the way it determines the report type to allow both ArcCheck and MapCheck, in any upper/lower case combination.

cutright commented 4 years ago

I think that's generally true. I'll give it a spin on MapCheck and ArcCheck reports here.