<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi Alfred,<div class="">Sorry for my late response, but KStars is still not my main profession :-)</div><div class=""><br class=""></div><div class="">I think we have a misunderstanding here. There are two checks about the guiding deviation:</div><div class="">1.) before capturing starts</div><div class="">2.) while capturing is running</div><div class=""><br class=""></div><div class="">We are currently talking only about 1.)</div><div class=""><br class=""></div><div class="">Currently (haven’t verified it 100%), 1.) is only executed when a new sequence (i.e. a new filter) is started. If a sequence contains more than one capture, it won’t be called before each capture starts.</div><div class=""><br class=""></div><div class="">I think that’s OK, since we have 2.) guarding during the entire capturing time. So if the threshold of both is equal, 1.) is not really necessary.</div><div class=""><br class=""></div><div class="">Conclusion: I would leave it as it is.</div><div class=""><br class=""></div><div class="">Does that make sense to you?</div><div class=""><br class=""></div><div class="">Cheers</div><div class="">Wolfgang<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">Am 19.08.2022 um 03:07 schrieb Hy Murveit <<a href="mailto:murveit@gmail.com" class="">murveit@gmail.com</a>>:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Wolfgang, <div class=""><br class=""></div><div class="">I don't think Alfred has it quite right, but here's my shot at what might be wrong.</div><div class=""><br class=""></div><div class="">In order for the pre-capture guide check to be enforced, prepareActions[ACTION_GUIDER_DRIFT] needs to be set to false.</div><div class="">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 class="">I believe that prepareLightFrameCapture is only called at the start of a job, and not before each capture, since:</div><div class=""><ul class=""><li class="">it is only called by SequenceJob::prepareCapture()</li><li class="">which is called only by Capture::preparePreCaptureActions() </li><li class="">which is called by both Capture::scriptFinished (I'm ignoring that, doesn't seem relevant) and Capture::prepareActiveJobStage2() </li><li class="">which is called by both Capture::scriptFinished (I'm ignoring that, doesn't seem relevant) and Capture::prepareActiveJobStage1() </li><li class="">which is only called by Capture::prepareJob</li></ul><div class="">and from its title, prepareJob() sounds like it is only called at the start of a job, not each individual capture.</div></div><div class=""><br class=""></div><div class="">Perhaps the bug is that there should be an else at this line:</div><div class=""><a href="https://invent.kde.org/education/kstars/-/blob/master/kstars/ekos/capture/sequencejobstate.cpp#L722" class="">https://invent.kde.org/education/kstars/-/blob/master/kstars/ekos/capture/sequencejobstate.cpp#L722</a></div><div class="">e.g.</div><div class="">else prepareActions[ACTION_GUIDER_DRIFT] = false;</div><div class=""><br class=""></div><div class="">Hy</div></div><br class=""><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" class="">amiga2000c@gmx.de</a> <<a href="mailto:amiga2000c@gmx.de" class="">amiga2000c@gmx.de</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br class="">
Despite the fact that I have absolutely no clue about C++, I had a look<br class="">
at the code. I think the relevant piece is "capture.cpp".<br class="">
<br class="">
A passage where 10 checks are being performed (6039-6121) before<br class="">
"everything is ready for capturing light frames" cought my attention.<br class="">
<br class="">
<br class="">
Step 6 is check guide deviation. Assuming guiding was suspended and the<br class="">
last measured guiding deviation was below the limit, this check is passed.<br class="">
Step 10 is check if guiding was suspended. If so, it is restarted and<br class="">
according to the comment it is assumed that "everything is ready for<br class="">
capturing light frames" then.<br class="">
<br class="">
IMO this is not the case. After resuming guiding, the "check guide<br class="">
deviation" test must be initiated again and only if it is passed,<br class="">
"everything is ready for capturing light frames", indeed.<br class="">
<br class="">
--<br class="">
<br class="">
In case what I wrote is complete rubbish, please excuse my clueless<br class="">
comment and ignore it.<br class="">
<br class="">
Regards<br class="">
Alfred<br class="">
</blockquote></div>
</div></blockquote></div><br class=""></div></body></html>