HomePhabricator
Diffusion RTTB a21335e3c940

Changed implementation to first extend the already existing precomputeValues…

Description

Changed implementation to first extend the already existing precomputeValues vector and aferwards compute all not already computed values

Details

Auditors
hentsch
Provenance
strubelpAuthored on Jun 30 2017, 12:09 PM
strubelpPushed on Jun 30 2017, 12:12 PM
Parents
rRTTB2b3cee1d10a4: added typedef to avoind along badly readable boost pointers
Branches
Unknown
Tags
Unknown

Event Timeline

hentsch added a subscriber: hentsch.

sehr schön, das sieht gut aus. Sind jetzt nur noch Kleinigkeiten ;)

/code/algorithms/rttbDoseStatisticsCalculator.h
182

typo: complex

187

hier auch ;)

/code/algorithms/rttbDoseToVolumeMeasureCollection.cpp
84

glaube, hier ist es passend, wenn man double imprecision berücksichtigt: also |t1-t2|<eps, wobei eps = 1e-5

/code/algorithms/rttbDoseToVolumeMeasureCollection.h
66

funktion sollte private sein

/code/algorithms/rttbDoseToVolumeMeasureCollectionCalculator.cpp
65

hier könnte man auch gut noch checken, ob das value (double imprecision beachten) bereits drin ist. Sonst wird es einfach doppelt berechnet.

/code/algorithms/rttbDoseToVolumeMeasureCollectionCalculator.h
64

typo: computed

/code/algorithms/rttbVolumeToDoseMeasureCollection.cpp
91

hier gilt natürlich dasselbe wie bei DoseToVolumeMeasureCollection

This commit now has outstanding concerns.Jun 30 2017, 2:22 PM
strubelp added inline comments.
/code/algorithms/rttbDoseToVolumeMeasureCollection.h
66

dann kann man aber nichtmehr von rttbDoseToVolumeMeasureCollectionCalculator drauf zugreifen

All concerns with this commit have now been addressed.Sep 28 2017, 9:00 AM