Closed michaelschwier closed 6 years ago
Merging #317 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #317 +/- ##
=======================================
Coverage 79.64% 79.64%
=======================================
Files 28 28
Lines 2398 2398
=======================================
Hits 1910 1910
Misses 488 488
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update e5c3934...9475369. Read the comment docs.
@michaelschwier I actually forgot about this! This should be quite helpful for another project.
These are a few classes I used to be able to conveniently put together measurement reports from custom data storage in a python batch processing script. The classes allow export to JSON which then can directly be processed by the tid1500 writer.
Currently these classes are still quite specific to my task (converting Prostate MRI Test-Retest Data/Measurements to DICOM). However I think the basic idea is nice and can be easily extended to be much more generic. The classes are mainly data structures with the same layout as the respective parts of the JSON meta file, plus some convenience methods (e.g. exporting as JSON, adding sub-items).
To make it more generic my idea would be to extend e.g. the Findings and MeasurementItem classes by proper factories that create all the typical objects by a few keywords (and also give the user a generic fallback to set all members of an object manually for cases not covered yet). In general: create convenient prepared classes so a user only has to provide the essential data/info and not worry about which special codes to populate them with.
Building this in python would also allow double use of this util library: