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