D29099: Make the Focus capture action include star detection

Eric Dejouhanet noreply at phabricator.kde.org
Wed Apr 22 16:05:49 BST 2020


TallFurryMan created this revision.
TallFurryMan added a reviewer: KStars.
Herald added a project: KDE Edu.
Herald added a subscriber: kde-edu.
TallFurryMan requested review of this revision.

REVISION SUMMARY
  This differential makes the Focus capture button state reflect exposure and detection.
  Before that change, the capture button reverted to the enable state after the frame was received but the source extraction continued in the background.
  The UI was therefore ready to accept user input, but at the same time unresponsive.
  The differential adjusts the UI tests to wait for the capture button to re-enable, instead of inserting a suitable delay before waiting.
  
  This differential also replaces an assertion generated when the tracking box is enabled in full-field detection, with an immediate disable of the tracking box.
  This differential also rewrites the tooltip displayed by the detection method drop-down, providing more detailed information.
  This differential also fixes the explicit assignment operator warnings.

TEST PLAN
  UI tests through kstars_ui_tests. Jenkins builds for warnings.

REPOSITORY
  R321 KStars

BRANCH
  improve__determine_star_detection_end (branched from master)

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

AFFECTED FILES
  Tests/kstars_ui/test_ekos_focus.cpp
  Tests/kstars_ui/test_ekos_focus.h
  kstars/ekos/focus/focus.cpp
  kstars/ekos/focus/focus.ui
  kstars/fitsviewer/fpackutil.c
  kstars/printing/legend.cpp
  kstars/printing/legend.h
  kstars/time/kstarsdatetime.cpp
  kstars/time/kstarsdatetime.h
  kstars/tools/modcalcvizequinox.cpp

To: TallFurryMan, #kstars
Cc: kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20200422/077e7e6c/attachment.html>


More information about the kde-edu mailing list