<table><tr><td style="">TallFurryMan edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-jlubopro5574ol5/">(Show Details)</a><br />TallFurryMan edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-5qzbpjysdzqhqm5/">(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;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- </span>Adds a type for the captured frame map<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">.<br />
- Updates captured frames map when a capture completes.<br />
- Removes use of general variables and prefer activeJob whenever possible in the capture module.<br />
- Fixes the issue with duplicated scheduler jobs using duplicated sequence jobs. (Issue was in the duration estimation, which was not properly consolidating captured frames for single scheduler jobs.)<br />
- Fixes evaluation of Hour Angle when checking meridian flip in the case DEC is +/-90 degrees (undefined HA).<br />
- Fixes unexpected slew when repeating a scheduler job with no pipeline step checked.<br />
- Fixes dbus error management, reworks function unParkMount to clarify state.<br />
- Fixes park/unpark looping in unparked/parked state, reworks checkMountParkingStatus</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Updates captured frames map when a capture completes</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Fix park/unpark looping in unparked/parked state, rework checkMountParkingStatus</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Removes use of general variables and prefer activeJob whenever possible in the</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Fix capture progress at 100% at the beginning of</span> captur<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">e module</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">e</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Fixes the issue with duplicated scheduler jobs using duplicated sequence jobs</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Fix incoherent use of target prefix in the capture path</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">(Issue was in the duration estimation, which was not properly consolidating</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Add a message in the Capture module when a</span> capture<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">d frames for single scheduler jobs.)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> aborts.</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Fixes evaluation of Hour Angle</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Fix signature management</span> when <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">checking meridian flip in the case DEC is +/-90 degrees (undefined HA)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">an hardcoded prefix is used in the sequence</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Fixes unexpected slew when repeating a scheduler job with no pipeline step checked</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Update duplicate job warning, incorrect when two different sequences have the same storage</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Fixes dbus error management, reworks function unParkMount to clarify state</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Fix no-step job looping on altitude check</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Fixes park/unpark looping in unparked/parked state, reworks checkMountParkingStatus</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Fix update of captured frames map for sequence jobs already complete</span>.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>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 />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Image count update in the Scheduler is still an issue, and is probably not just picking a signal and increasing a count.<br />
<br />
There is a spurious folder created with the name of the capture prefix. This is a regression from the signature fix for the prefix issue. Fairly minimal, but not professional.<br />
<br />
</span>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.<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>