<div dir="ltr">Wolfgang, <div><br></div><div>I don't think Alfred has it quite right, but here's my shot at what might be wrong.</div><div><br></div><div>In order for the pre-capture guide check to be enforced, prepareActions[ACTION_GUIDER_DRIFT] needs to be set to false.</div><div>As far as I can tell, during capture execution it is only set to false in SequenceJobState::prepareLightFrameCapture in sequencejobstate.cpp line 50</div><div>I believe that prepareLightFrameCapture is only called at the start of a job, and not before each capture, since:</div><div><ul><li>it is only called by SequenceJob::prepareCapture()</li><li>which is called only by Capture::preparePreCaptureActions() </li><li>which is called by both Capture::scriptFinished (I'm ignoring that, doesn't seem relevant) and Capture::prepareActiveJobStage2() </li><li>which is called by both Capture::scriptFinished (I'm ignoring that, doesn't seem relevant) and Capture::prepareActiveJobStage1() </li><li>which is only called by Capture::prepareJob</li></ul><div>and from its title, prepareJob() sounds like it is only called at the start of a job, not each individual capture.</div></div><div><br></div><div>Perhaps the bug is that there should be an else at this line:</div><div><a href="https://invent.kde.org/education/kstars/-/blob/master/kstars/ekos/capture/sequencejobstate.cpp#L722">https://invent.kde.org/education/kstars/-/blob/master/kstars/ekos/capture/sequencejobstate.cpp#L722</a></div><div>e.g.</div><div>else prepareActions[ACTION_GUIDER_DRIFT] = false;</div><div><br></div><div>Hy</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 18, 2022 at 3:32 PM <a href="mailto:amiga2000c@gmx.de">amiga2000c@gmx.de</a> <<a href="mailto:amiga2000c@gmx.de">amiga2000c@gmx.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Despite the fact that I have absolutely no clue about C++, I had a look<br>
at the code. I think the relevant piece is "capture.cpp".<br>
<br>
A passage where 10 checks are being performed (6039-6121) before<br>
"everything is ready for capturing light frames" cought my attention.<br>
<br>
<br>
Step 6 is check guide deviation. Assuming guiding was suspended and the<br>
last measured guiding deviation was below the limit, this check is passed.<br>
Step 10 is check if guiding was suspended. If so, it is restarted and<br>
according to the comment it is assumed that "everything is ready for<br>
capturing light frames" then.<br>
<br>
IMO this is not the case. After resuming guiding, the "check guide<br>
deviation" test must be initiated again and only if it is passed,<br>
"everything is ready for capturing light frames", indeed.<br>
<br>
--<br>
<br>
In case what I wrote is complete rubbish, please excuse my clueless<br>
comment and ignore it.<br>
<br>
Regards<br>
Alfred<br>
</blockquote></div>