<table><tr><td style="">murveit updated this revision to Diff 79254.<br />murveit added a comment.
</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><div><p>Reverted back to CENTROID as findStars default, still use SEP for mark stars, added hfr to status</p>

<p>After Eric comment, I reverted my change to the findStars definition, so it will default to CENTROID,<br />
but I kept the functionality of using SEP to detect stars for Mark Stars (it explicitly calls for that.</p>

<p>Here's my rationale. First of all, CENTROID does work for this. I tried with my own subs, and using the simulator<br />
with whatever (probably near default) settins I currently have, and in both cases I get no star detections with<br />
CENTROID and 100 for SEP. See <a href="https://photos.app.goo.gl/taXsZXnVpwtqsSvp8" class="remarkup-link" target="_blank" rel="noreferrer">https://photos.app.goo.gl/taXsZXnVpwtqsSvp8</a> which was "with ALGORITH_SEP", and <br />
<a href="https://photos.app.goo.gl/qz3gnBdk7GtMw5MEA" class="remarkup-link" target="_blank" rel="noreferrer">https://photos.app.goo.gl/qz3gnBdk7GtMw5MEA</a> which was with ALGORITHM_CENTROID.</p>

<p>Secondly, now as it is, it will just affect Mark Star, which was busted previously, so this won't hurt anyone, and now works,</p>

<p>Thirdly, I can easily speed up this application of SEP by at least a factor of 2, and that was my plan for a follow up PR,<br />
which I'd be happy to show you, but...</p>

<p>Fourthly, I just heard from Jasem that Rob is removing SEP and probably all the other algorithms in favor of SEXTRACTOR.<br />
So this is probably moot, and IMHO we should add this as I have it, and then when Rob's changes are ready, move it all to SEXTRACTOR.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D28477?vs=79026&id=79254">https://phabricator.kde.org/D28477?vs=79026&id=79254</a></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.cpp<br />
kstars/fitsviewer/fitsviewer.cpp</div></div></div><br /><div><strong>To: </strong>murveit, mutlaqja, TallFurryMan<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>