<table><tr><td style="">TallFurryMan edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-lccduldyfunw3dl/">(Show Details)</a><br />TallFurryMan edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-yngpbiuvw5r5y6c/">(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/D14684">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>Removes use of general variables and prefer activeJob whenever possible in the capture module.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Fixes the issue with duplicated scheduler jobs using duplicated sequence jobs.<br />
(Issue was in the duration estimation, which was not properly consolidating captured frames for single scheduler jobs.)</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Attempts to fix the issue with duplicated scheduler jobs using duplicated sequence jobs</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Fixes evaluation of Hour Angle when checking meridian flip in the case DEC is +/-90 degrees (undefined HA)</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Issue was in the duration estimation,</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Fixes unexpected slew when repeating a scheduler job with no pipeline step checked.<br />
<br />
There are no additional abort detections, so Scheduler may still get stuck.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">which was not properly consolidating captured frames for single s</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">A later change will introduce timeouts for S</span>cheduler <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">jobs</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">steps</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Use as many tests vectors as possible, in various configurations.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">There are no additional abort detections</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- simple_test tested OK<br />
- simple_test_no_twilight tested OK<br />
- duplicated_scheduler_jobs_no_twilight tested OK<br />
- duplicated_scheduler_jobs_duplicated_sequence_jobs_no_twilight tested OK<br />
- complex_job tested OK<br />
<br />
Note there is an issue with parking/unparking when executing jobs one after the other. When the second job tries to unpark the telescope simulator, it loops on "Mount parked". This is under control, to be pushed later on.<br />
<br />
Message "Manual scheduled focusing is not supported" is still annoying when running test vectors in sequence, to be investigated and eventually removed very soon.<br />
<br />
Guider is doing weird things when used with the simulator. It detects offset on the guide star although the simulator always produce the same image, only noise is different from frame to frame, and this seems to cause issues with drift calculation. Or the settings of my guider page are completely off... However, the guider is able to produce a guiding deviation that exceeds the bounds acceptable for exposure, and capture suspends.<br />
Now this is still an issue in the last differential: capture module suspends for 60 seconds waiting for the guider, guider never really asserts return to bounds</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">so Scheduler may still get stuck</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">and capture module never asserts the suspend timer neither</span>. <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">A later change will introduce timeouts for Scheduler steps</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Ooof</span>.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14684">https://phabricator.kde.org/D14684</a></div></div><br /><div><strong>To: </strong>TallFurryMan, mutlaqja, wreissenberger<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>