Added VolumeToDose Measure/Calculator and implemented Dx.
Concern Raised8b8a664bc8d0
Actions

Description

Added VolumeToDose Measure/Calculator and implemented Dx.

Auditors

Provenance

• strubelp	Authored on Jun 2 2017, 10:48 AM
• strubelp	Pushed on Jun 9 2017, 1:43 PM

so gut wie alles kleinere Sachen

/code/algorithms/rttbDxVolumeToDoseMeasureCalculator.h
25–26	beide includes sind schon in der Basisklasse drin
33	bitte auf Groß/Kleinschreibung achten. Visual Studio frisst das, bei GCC gibt es da Probleme.
46	bei komplexen Datentypen (const std::vector<double>&) ist const referenz eine gute idee, bei einfachen (const VolumeType& = const double&) ist es unüblich und bringt keine Performance vorteile.
51	hier fände ich DoseTypeGy computeSpecificValue(VolumeType xAbsolute) const besser
/code/algorithms/rttbVolumeToDoseMeasure.h
25	sehe gerade nicht, dass vector gebraucht wird.
28	ist evtl unnötig
51	Macht es Sinn, den enum DoseStatistics::complexStatistics in diese Klasse umzuziehen?
52	get*() const geht immer ;)
/code/algorithms/rttbVolumeToDoseMeasureCalculator.h
50	braucht man volume nicht auch in den subklassen?
58	funktion kann const sein. Und: Wir müssen etwas mit der Benennung aufpassen. Bei measure würde ich erstmal einen skalaren Wert vermuten. Wir haben hier aber im prinzip eine measureCollection von Zuordnungen: f(x) = x, z.B. D5 = 23.
59	siehe Kommentar bei DxVolumeToDoseCalculator (hier fände ich DoseTypeGy computeSpecificValue(VolumeType xAbsolute) const besser)

This commit now has outstanding concerns.Jun 12 2017, 1:34 PM

• strubelp marked 10 inline comments as done.Jun 12 2017, 4:40 PM

• strubelp added inline comments.

/code/algorithms/rttbVolumeToDoseMeasureCalculator.h
50	nein, da man immer "_precomputeVolumeValues.at(i) * _volume" übergibt

• strubelp marked 3 inline comments as done.Jun 13 2017, 12:05 PM

This commit no longer exists in the repository.