<table><tr><td style="">mutlaqja added a comment.
</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/D15546">View Revision</a></tr></table><br /><div><div><p>SchedulerJob::STAGE_SLEWING is handled in an event driven fashion in setMountStatus(). I don't think we should be using slewStatus() for this, the mount overall status should suffice (ISD::Telescope::STATUS). The reason I moved SLEW_RESLEWING to checkJob status is that in the event when reslewing is not necessary (i.e. mount is already at location so when you tell it to reslew, its state would remain as is (TRACKING), and therefore won't emit a signal that we can catch in setMountStatus) then it uses polling to check mount status and then proceed. But I agree with your dome check changes they make sense. Right now, STAGE_SLEWING is checked twice in your code, in checkJobStatus and in setMountStatus</p></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/D15546">https://phabricator.kde.org/D15546</a></div></div><br /><div><strong>To: </strong>TallFurryMan, mutlaqja, wreissenberger<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>