<table><tr><td style="">murveit created this revision.<br />murveit added reviewers: mutlaqja, TallFurryMan.<br />Herald added a project: KDE Edu.<br />Herald added a subscriber: kde-edu.<br />murveit requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28477">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>After this fix, to display the HFR value in the fitsviewer statistics, one needs to first<br />
set the fitsviewer's view menu 'Mark Stars' option, and then, the next time an image is captured,<br />
the HFR will be computed and the Statistics Table will show the HFR value.<br />
The HFR would be computed for the current image, but it wouldn't be updated in the table.<br />
One cannot set that view option, though, until the fitsviewer is opened, which is normally<br />
after a capture. Obviously, this isn't a great UI. A subsequent PR should create a way for this<br />
to be set by default across sessions.</p>

<p>Before this fix, the Statistics Table would never display the HFR, whether it was computed or not.</p>

<p>This fix moves the display to after the computation, instead of before (which is why it wasn't displayed).<br />
This fix also changes the algorithm used to SEP instead of CENTROID. At least in the simulator, CENTROID<br />
was not finding any stars. This should be the default algorithm anyway, as SEP is much better.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Run the simulator. Set a capture job, e.g. 1s of a red filter. Note HFR is -1 in the statistics table.<br />
In the fitsviewer menu, go to View -> MarkStars.<br />
Reset status of the capture job, run it again. You should now get a reasonable HFR value in the statistics table.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>hfr-display-fix (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28477">https://phabricator.kde.org/D28477</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kstars/fitsviewer/fitstab.cpp<br />
kstars/fitsviewer/fitsview.h</div></div></div><br /><div><strong>To: </strong>murveit, mutlaqja, TallFurryMan<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>