HomePhabricator
Diffusion RTTB 4eaf5ada94a7

Fixed typos and added check for double imprescision

Description

Fixed typos and added check for double imprescision

Details

Auditors
hentsch
Provenance
strubelpAuthored on Jun 30 2017, 3:14 PM
strubelpPushed on Jun 30 2017, 3:21 PM
Parents
rRTTBfbec895c3da2: Removed deprecated functions in MaskVoxel Signed-off-by: Clemens Hentschke <c.
Branches
Unknown
Tags
Unknown

Event Timeline

hentsch added a subscriber: hentsch.
hentsch added inline comments.
/code/algorithms/rttbDoseToVolumeMeasureCollection.cpp
84

in baseType.h ist const double errorConstant = 1e-5; definiert, das kannst du wiederverwenden :)

86
/code/algorithms/rttbDoseToVolumeMeasureCollectionCalculator.h
86

funktion kann const sein. Würde sie auch findKey(double, std::map<>) nennen.
Ist natürlich unschön, wenn sie in beiden Klassen auftaucht. In *MeasureCollection hat sie aber auch wenig zu suchen.
Es ist generisch genug, dass sie potential für Wiederverwendung hat. Was hältst du davon, wenn wir im RTTBCore Modul eine rttbUtils datei aufmachen mit static methoden?

This commit now has outstanding concerns.Jul 4 2017, 10:56 AM
All concerns with this commit have now been addressed.Sep 28 2017, 9:00 AM