Fix HFR calculation, add HFR on focus view, adjust focus UI.

Authored by mutlaqja on Sep 19 2019, 10:26 AM.

Description

Fix HFR calculation, add HFR on focus view, adjust focus UI.

Summary:
Fixed HFR calculation in the FITSData class, which was accumulating integer instead of double values.
This led to incorrect consolidation of HFR over multiple stars in the focus procedure and incoherent results in full-frame mode.

In order to debug, added HFR values directly in the FITS view, next to circular star marks.
The HFR values are now displayed only for the focus module, they do not appear in the regular FITS viewer.

Took the opportunity to adjust the Focus module UI, so that buttons are bigger and a few UI hiccups are resolved.

Notes:

At the time of this differential, the FITS viewer and the Focus module do not detect the same amount of stars.
Apparently, the "JMIndex" in the case of the FITS viewer is taken from the histogram, and equals 0, causing the detection threshold to be computed from the frame mean ADU, and set too high.
The "JMIndex" in the case of the Focus module is left to its default value 100, causing the detection threshold to be computed from the frame standard deviation, and set properly.

On another subject, the CCD Simulator seems to produce the same SNR even as the optical simulation approaches the optimal focus point.
This causes rendered stars to be smaller, and thus to be missed by the FITS detector. In other words, the more focused the frame is, the less stars are detected.
Because of this problem, the focus procedure sometimes fails with the CCD Simulator.

Test Plan:

  • Run an autofocus procedure using the CCD Simulator or a real CCD night test so that a dozen of stars are detected.
  • Verify the HFR consolidated for a particular focuser position is the mean value of HFR registered for detected stars.
  • Verify the procedure is overall more stable than before the change.

Reviewers: mutlaqja, alexcherney, wreissenberger

Reviewed By: mutlaqja, wreissenberger

Subscribers: kde-edu

Tags: KDE Edu

Differential Revision: https://phabricator.kde.org/D23892

Details

Committed
mutlaqjaSep 19 2019, 10:26 AM
Reviewer
mutlaqja
Differential Revision
D23892: Fix HFR calculation, add HFR on focus view, adjust focus UI.
Parents
R321:f7a06d684a5c: Use reasonable minimum that does not squish the GUI
Branches
Unknown
Tags
Unknown