<table><tr><td style="">murveit created this revision.<br />murveit added reviewers: mutlaqja, lancaster, 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/D28767">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This PR should improve the HFR calculations with SEP for the focus module and for displaying HFR for those that want to track it in subs.<br />
It is not the end-all, but should be an improvement. There is still work to do.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Updated and added to tests for the fitssepdetector.cpp</li>
<li class="remarkup-list-item">Removed saturated stars from the HFR calculation, if possible.</li>
<li class="remarkup-list-item">Sped up the SEP star detector 2X when used with full exposures.</li>
<li class="remarkup-list-item">Improved SEP star detector quality, especially for focus/hfr, by (1) changing the deblend constant, and (2) removing the largest stars</li>
<li class="remarkup-list-item">Fixed two bugs in the SEP star detector (1) major issue, ushorts being treated as shorts, and (2) a memory leak for (rare) float images</li>
<li class="remarkup-list-item">Added logging for HFR when calculated</li>
<li class="remarkup-list-item">Fixed bug so that Mark Stars now works</li>
<li class="remarkup-list-item">Improved drawing of circles for Mark Stars, now uses star radius, instead of flakey star width.</li>
<li class="remarkup-list-item">Updated the fitsviewer status to include the HFR when calculated</li>
<li class="remarkup-list-item">Increased a threshold in SEP extract.c which previously would fail on images, e.g. like <a href="https://phabricator.kde.org/M42" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">M42</a>, where there was a lot of area over background.</li>
</ul>

<p>Not done, but related:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Should improve the MarkStars UI so that HFR could be calculated without star annotation on display, and could turn on on first image.</li>
<li class="remarkup-list-item">API to star detection methods should be generalized (should wait on rlancaste's investigations of SEP best practices)</li>
<li class="remarkup-list-item">Some display of HFR trends.</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p>Basic test added to testfitsdata for SEP, run that.<br />
Load a .fits file into fitsviewer. Select view->MarkStars, for a full-exposure sub, you should see ~100 stars circled with an HFR displayed in the status bar.<br />
Load another image and the HFR should continue to be displayed.<br />
Similarly, one could make a capture sequence job with, say 20 5s simulator exposures. After the first exposure completes and the fitsviewer displays<br />
the image, go to the view menu and turn on Mark Stars, then the rest of the images should display with stars circled and HFR displayed in the status bar.<br />
This way of turning on the star/HFR detection should be improved in a future PR.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>sep-focus-improvements (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28767">https://phabricator.kde.org/D28767</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>Tests/fitsviewer/testfitsdata.cpp<br />
kstars/fitsviewer/fitsdata.cpp<br />
kstars/fitsviewer/fitssepdetector.cpp<br />
kstars/fitsviewer/fitstab.cpp<br />
kstars/fitsviewer/fitsview.cpp<br />
kstars/fitsviewer/fitsviewer.cpp<br />
kstars/fitsviewer/sep/extract.c</div></div></div><br /><div><strong>To: </strong>murveit, mutlaqja, lancaster, TallFurryMan<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>