Skip to content

Commit c140380

Browse files
committed
Merge branch 'dev' into rc/v3.2.0
2 parents 046b363 + cfb49f0 commit c140380

File tree

2 files changed

+53
-42
lines changed

2 files changed

+53
-42
lines changed

matRad/scenarios/matRad_ImportanceScenarios.m

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@
5959
matRad_cfg = MatRad_Config.instance();
6060
matRad_cfg.dispError('Invalid number of setup grid points, needs to be a positive scalar!');
6161
end
62-
this.numOfSetupGridPoints = inumGridPoints;
62+
this.numOfSetupGridPoints = numGridPoints;
6363
this.updateScenarios();
6464
end
6565

@@ -69,7 +69,7 @@
6969
matRad_cfg = MatRad_Config.instance();
7070
matRad_cfg.dispError('Invalid number of range grid points, needs to be a positive scalar!');
7171
end
72-
this.numOfRAngeGridPoints = inumGridPoints;
72+
this.numOfRangeGridPoints = numGridPoints;
7373
this.updateScenarios();
7474
end
7575
end

test/scenarios/test_importanceScenarios.m

Lines changed: 51 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -143,47 +143,58 @@
143143
end
144144

145145

146-
function test_importanceScenarioCombineRange
146+
function test_importanceScenarioCombineRange
147147

148-
model = matRad_ImportanceScenarios();
149-
150-
assertExceptionThrown(@() helper_assignmentTest(model,'combineRange','hello'),'matRad:Error');
151-
assertTrue(model.combineRange);
152-
153-
nRangeScen = model.totNumRangeScen;
154-
155-
model.combineRange = false;
156-
assertFalse(model.combineRange);
157-
assertEqual(model.totNumRangeScen,nRangeScen^2);
158-
assertEqual(model.totNumScen,model.totNumRangeScen - 1 + model.totNumShiftScen);
159-
160-
function test_importanceScenarioShiftCombinations
161-
162-
model = matRad_ImportanceScenarios();
163-
164-
assertExceptionThrown(@() helper_assignmentTest(model,'combinations','hello'),'matRad:Error');
165-
assertEqual(model.combinations,'none');
166-
167-
nSetupPoints = model.numOfSetupGridPoints;
168-
nRangePoints = model.numOfRangeGridPoints;
148+
model = matRad_ImportanceScenarios();
169149

170-
model.combinations = 'shift';
171-
assertEqual(model.combinations,'shift');
172-
assertEqual(model.totNumShiftScen,nSetupPoints^3);
173-
assertEqual(model.totNumScen,model.totNumRangeScen - 1 + model.totNumShiftScen);
150+
assertExceptionThrown(@() helper_assignmentTest(model,'combineRange','hello'),'matRad:Error');
151+
assertTrue(model.combineRange);
174152

175-
model.combinations = 'all';
176-
assertEqual(model.combinations,'all');
177-
assertEqual(model.totNumShiftScen,nSetupPoints^3);
178-
assertEqual(model.totNumRangeScen,nRangePoints);
179-
assertEqual(model.totNumScen,model.totNumRangeScen * model.totNumShiftScen);
180-
181-
model.combineRange = false;
182-
assertEqual(model.totNumShiftScen,nSetupPoints^3);
183-
assertEqual(model.totNumRangeScen,nRangePoints^2);
184-
assertEqual(model.totNumScen,model.totNumRangeScen * model.totNumShiftScen);
153+
nRangeScen = model.totNumRangeScen;
154+
155+
model.combineRange = false;
156+
assertFalse(model.combineRange);
157+
assertEqual(model.totNumRangeScen,nRangeScen^2);
158+
assertEqual(model.totNumScen,model.totNumRangeScen - 1 + model.totNumShiftScen);
185159

186-
model.combinations = 'shift';
187-
assertEqual(model.totNumShiftScen,nSetupPoints^3);
188-
assertEqual(model.totNumRangeScen,nRangePoints^2);
189-
assertEqual(model.totNumScen,model.totNumRangeScen + model.totNumShiftScen - 1);
160+
function test_importanceScenarioShiftCombinations
161+
162+
model = matRad_ImportanceScenarios();
163+
164+
assertExceptionThrown(@() helper_assignmentTest(model,'combinations','hello'),'matRad:Error');
165+
assertEqual(model.combinations,'none');
166+
167+
nSetupPoints = model.numOfSetupGridPoints;
168+
nRangePoints = model.numOfRangeGridPoints;
169+
170+
model.combinations = 'shift';
171+
assertEqual(model.combinations,'shift');
172+
assertEqual(model.totNumShiftScen,nSetupPoints^3);
173+
assertEqual(model.totNumScen,model.totNumRangeScen - 1 + model.totNumShiftScen);
174+
175+
model.combinations = 'all';
176+
assertEqual(model.combinations,'all');
177+
assertEqual(model.totNumShiftScen,nSetupPoints^3);
178+
assertEqual(model.totNumRangeScen,nRangePoints);
179+
assertEqual(model.totNumScen,model.totNumRangeScen * model.totNumShiftScen);
180+
181+
model.combineRange = false;
182+
assertEqual(model.totNumShiftScen,nSetupPoints^3);
183+
assertEqual(model.totNumRangeScen,nRangePoints^2);
184+
assertEqual(model.totNumScen,model.totNumRangeScen * model.totNumShiftScen);
185+
186+
model.combinations = 'shift';
187+
assertEqual(model.totNumShiftScen,nSetupPoints^3);
188+
assertEqual(model.totNumRangeScen,nRangePoints^2);
189+
assertEqual(model.totNumScen,model.totNumRangeScen + model.totNumShiftScen - 1);
190+
191+
function test_importanceScenariosPropertySetters
192+
model = matRad_ImportanceScenarios();
193+
194+
model.numOfSetupGridPoints = 5;
195+
assertEqual(model.numOfSetupGridPoints,5);
196+
197+
model.numOfRangeGridPoints = 5;
198+
assertEqual(model.numOfRangeGridPoints,5);
199+
200+

0 commit comments

Comments
 (0)