Hotfixes for the Focus module
ClosedPublic

Authored by TallFurryMan on Apr 8 2020, 10:03 PM.

Details

Summary

This differential is a hotfix for the recent work in the Focus module.
One additional class, FITSSkyObject, was extracted and put in its own source files.
The JMIndex parameter of the Centroid detection was not provided to the source extraction procedure.
The Star Profile Viewer missed a few consts in its management of FITS data buffer, which caused some Jenkins builds to fail on SUSE and FreeBSD.

Test Plan

Run kstars_ui_tests

Diff Detail

Repository
R321 KStars
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
TallFurryMan requested review of this revision.Apr 8 2020, 10:03 PM
TallFurryMan created this revision.

A few things I wanted to add but will delay:

  • UI does not reflect the use of SEP and Centroid for initial detection, nor for full-field analysis.
  • Due to the first point, the tests are not properly verifying the Threshold and Gradient algorithms because the full-field analysis option is set!
  • UI does a weird toggle on the capture button when preparing the screen, and re-enables said button *before* the star extraction completes (that code block is tricky to refactor).
  • Due to the first point, the tests need to wait an arbitrary duration to verify the detection results.
mutlaqja accepted this revision.Apr 8 2020, 11:16 PM
This revision is now accepted and ready to land.Apr 8 2020, 11:16 PM
This revision was automatically updated to reflect the committed changes.