<div dir="ltr">Hi Simon & RTK developers & the RTK community,<div><br></div><div>I have a question about how delta is calculated in rtkParkerShortScanImageFilter.txt.</div><div><br></div><div>The itFirstAngle and the itLastAngle were used to find the first and last angular values for calculating delta.</div><div><br></div><div>However I noticed that at line 93 itFirstAngle is acquired by:</div><div><br></div><div><div>  itFirstAngle = sortedAngles.find(rotationAngles[maxAngularGapPos]);</div><div>  itFirstAngle = (++itFirstAngle==sortedAngles.end())?sortedAngles.begin():itFirstAngle;</div><div>  itFirstAngle = (++itFirstAngle==sortedAngles.end())?sortedAngles.begin():itFirstAngle;</div></div><div><br></div><div>Why are the second and third lines duplicated? Wouldn't this point to the "second" angle instead?</div><div><br></div><div>And also for itLastAngle at line 99:</div><div><br></div><div><div>  itLastAngle = sortedAngles.find(rotationAngles[maxAngularGapPos]);</div><div>  itLastAngle = (itLastAngle==sortedAngles.begin())?--sortedAngles.end():--itLastAngle;</div></div><div><br></div><div>Why is the second line needed here? Wouldn't this point to the "second last" angle instead? (Since maxAngularGapPos was caluclated with GetAngularGapsWithNext).</div><div><br></div><div>I did test this with a 0-200 deg short scan, and found that firatAngle and lastAngle weren't equal to 0 and 200, but were calculated to be the second and second last angles instead.</div><div>Is there any reason for this implementation?</div><div><br></div><div>Thank you.</div><div><br></div><div>Cheers,</div><div>Andy</div></div>