D15546: Fix Scheduler slewing stage andwq dome interaction.

[Jasem Mutlaq]

mutlaqja added a comment.


  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

REPOSITORY
  R321 KStars

REVISION DETAIL
  https://phabricator.kde.org/D15546

To: TallFurryMan, mutlaqja, wreissenberger
Cc: kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180916/728efd41/attachment.html>