D15436: Remove prevention of autofocus when no prior focus was done, add warnings.

Eric Dejouhanet noreply at phabricator.kde.org
Tue Sep 11 21:15:12 BST 2018


TallFurryMan created this revision.
TallFurryMan added reviewers: mutlaqja, wreissenberger.
Herald added a project: KDE Edu.
Herald added a subscriber: kde-edu.
TallFurryMan requested review of this revision.

REVISION SUMMARY
  This differential removes a safeguard on preliminary focus which is not always needed in the current state of the Focus module.
  The original code prevented the auto-focus method from starting without prior manual focus done at least once.
  This safeguard was here to prevent the situation where the optical train would be too far from the focal plane to allow star detection.
  In any case, it does not solve any particular problem, and gets in the way of the Scheduler when it starts the observatory by itself.
  
  This differential also adds a warning log about having a periodic or HFR refocus procedure set in Capture while no Focus step is set for the Scheduler job.

TEST PLAN
  Close and reopen KStars, create a very simple scheduler job with only the focus step selected, and execute it.
  Without the change, the focus step outputs the message "Manual scheduler focusing is not supported. Run autofocus process before trying again.".
  With the change, the focus step doesn't output that message and proceeds to auto-focus.

REPOSITORY
  R321 KStars

BRANCH
  improve__allow_autofocus_with_no_prior_focus (branched from master)

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

AFFECTED FILES
  kstars/ekos/capture/capture.cpp
  kstars/ekos/scheduler/scheduler.cpp

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/20180911/743b336f/attachment.html>


More information about the kde-edu mailing list