<table><tr><td style="">wreissenberger created this revision.<br />wreissenberger added reviewers: mutlaqja, chrisrowland, TallFurryMan, murveit.<br />Herald added a project: KDE Edu.<br />Herald added a subscriber: kde-edu.<br />wreissenberger requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D29836">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The capture module had two entry points for doing pre-capture checks: one when a new sequence is started and one for continuing to capture within a sequence. This creates several problems and unexpected behaviour. For example, focusing and dithering did only work for continuing a sequence.</p>

<p>With this diff, both starting a new sequence and continuing a sequence runs through the same check before starting to capture a new frame.</p>

<p>All changes are related to light frames only, capturing darks, bias and flat frames is (hopefully) untouched.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>This change touches the core of light frames capturing and requires intense testing. The tricky thing here is that the capture module handles by far more than pure frame capturing: it interacts with guiding and focusing, it starts dithering and interaction with the meridian flip, capturing may be started standalone or from the scheduler. At least the following needs to be tested:</p>

<p>General setup: mount is tracking, execute all tests for multiple Lum and 1xLRGB to ensure, that both in-sequence and new sequence.</p>

<p>Scenario 1: standalone capturing, no meridian flip</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">simple capture sequence with filter changes</li>
<li class="remarkup-list-item">abort capturing</li>
<li class="remarkup-list-item">capture with guiding and dithering</li>
<li class="remarkup-list-item">capture with guiding, star lost --> capture should abort after timeout</li>
<li class="remarkup-list-item">capture with focusing</li>
<li class="remarkup-list-item">capture with focusing, no stars found during focusing --> capture should abort after timeout</li>
</ul>

<p>Scenario 2: standalone capturing, mount crossing the meridian</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">all tests of scenario 1</li>
<li class="remarkup-list-item">capture with aligning</li>
<li class="remarkup-list-item">capture with aligning, no stars found during aligning --> capture should abort</li>
<li class="remarkup-list-item">abort capture during meridian flip waiting --> meridian flip should start after capture aborted</li>
<li class="remarkup-list-item">abort guiding during meridian flip waiting --> guiding should be restarted after meridian flip</li>
</ul>

<p>Scenario 3: capturing from a schedule</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">all tests of scenario 1, check for restart attempts when problems occured</li>
<li class="remarkup-list-item">all tests of scenario 2, check for restart attempts when problems occured</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>capture_refactor_start_new_image</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29836">https://phabricator.kde.org/D29836</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kstars/ekos/capture/capture.cpp<br />
kstars/ekos/capture/capture.h<br />
kstars/ekos/focus/focus.cpp<br />
kstars/ekos/mount/mount.cpp</div></div></div><br /><div><strong>To: </strong>wreissenberger, mutlaqja, chrisrowland, TallFurryMan, murveit<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>