<table><tr><td style="">wreissenberger edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-cmwjm5siticqwws/">(Show Details)</a><br />wreissenberger edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-sef2x4hbv4nbnfa/">(Show Details)</a>
</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/D25105">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">Restarting guiding after a meridian flip currently currently only happens, when the guider is in the guiding mode. For example, in the case that the meridian is crossed while dithering, the capture module does not recognize this as part of the guiding procedure and hence does not restart guiding as soon as the meridian flip is finished. This change fixes this and considers dithering, but also re-acquiring and calibrating as an indication that guiding should resume after meridian flip.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">As a side effect, this fixes also the situation when dithering fails in the middle of a capture sequence. Without this fix, a failed dithering is ignored and capturing resumes without restart guiding.<br />
<br />
</span>Additionally, a possible a weakness that may cause null pointer exceptions in starting focus execution is fixed. And last but not least, some dead code has been eliminated.<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
<br />
And last but not least, dead code is removed.</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Test case 1: </span>Enable dithering, move the scope close but east of the meridian and start a capturing sequence. In order to ensure, that the meridian is crossed while dithering, run kstars inside the debugger and set a breakpoint before the call of dithering.<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> Check, if after the meridian flip guiding is restarted.<br />
<br />
Test case 2: Enable dithering and create a schedule that uses a capture sequence with more than one capture. Start the schedule and interrupt guiding after the first capture while dithering.</span> Check,<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> if capturing and guiding resume.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">if after the meridian flip</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Test this with and without the option for aborting</span> guiding <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">is restarted</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">after dither failure set</span>.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>meridian_flip_bugfix_restart_guiding</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25105">https://phabricator.kde.org/D25105</a></div></div><br /><div><strong>To: </strong>wreissenberger, mutlaqja, lancaster<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>