Implemented way to trigger the computatuin of additional Dx dose statistic…
Auditedea83d641f1b3
Actions

Description

Implemented way to trigger the computatuin of additional Dx dose statistic values

Auditors

Provenance

• strubelp	Authored on Jun 29 2017, 11:21 AM
• strubelp	Pushed on Jun 29 2017, 3:42 PM

Parents

Branches

Unknown

Tags

Unknown

hentsch raised a concern with this commit.Jun 30 2017, 9:22 AM

hentsch added a subscriber: hentsch.

hentsch added inline comments.

/code/algorithms/rttbDoseStatistics.h
29	hier brauchst du den include nicht, im cpp schon
/code/algorithms/rttbDoseStatisticsCalculator.h
182	das ist ja eher ein addPrecomputeDoseValuesAndComputeTheirValues (eine fuktion, aber zwei aufgaben). Lieber addPrecomputeDoseValues() + calculateDoseStatistics() aufrufen (aber keins der bisherigen passt). Man könnte beispielsweise ein recalculateDoseStatistics() implementieren, das weiß, dass schon Werte berechnet wurden.
/code/algorithms/rttbDoseToVolumeMeasureCollectionCalculator.h
65	auch hier wieder: besser wäre addlPrecomputeDoseValues(). Fände es auch ok, wenn man explizit nochmal compute() aufrufen müsste
/code/algorithms/rttbVolumeToDoseMeasureCollectionCalculator.cpp
39–45	das würde ich genau anders rum machen. In compute() sollte immer die funktionalität stecken.
57	hier müsste dann noch ein check rein, ob es schon da ist oder nicht.